[llvm-commits] [PATCH][asan] %zu/%zx in asan-internal Printf()

[Kostya Serebryany]

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.
Thanks!

--kcc


>
> >
> > 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
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120321/51e88e47/attachment.html>