[llvm-commits] [PATCH][asan] %zu/%zx in asan-internal Printf()
kcc at google.com
Wed Mar 21 09:24:58 CDT 2012
On Tue, Mar 20, 2012 at 11:24 PM, Evgeniy Stepanov <
eugeni.stepanov at gmail.com> wrote:
> On Wed, Mar 21, 2012 at 2:35 AM, Joerg Sonnenberger
> <joerg at britannica.bec.de> wrote:
> > On Tue, Mar 20, 2012 at 09:23:57AM -0700, Kostya Serebryany wrote:
> >> The patch itself is fine, but I still don't understand why we need it.
> >> Is that just to avoid confusion? (%zu confuses me much more than %lu)
> >> Or you expect this to be actually required in some cases (or on some
> >> platforms)?
> > Win64 is IL32LL64, isn't it?
> Yes. You can not print a pointer with "%lu" there. You could with our
> implementation of Printf, and that's exactly the sort of confusion I'm
> trying to fix.
ok, please commit.
> > Joerg
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits