[compiler-rt] r230019 - [ASan] Make the argument of '__sanitizer_annotate_contiguous_container' is not aligned error message easier to understand
Timur Iskhodzhanov
timurrrr at google.com
Wed Feb 25 09:11:35 PST 2015
Macros are hard :(
Relanded as r230506, thanks for the help!
On Tue Feb 24 2015 at 11:21:31 PM Renato Golin <renato.golin at linaro.org>
wrote:
> I found the problem...
>
> On 20 February 2015 at 15:45, Timur Iskhodzhanov <timurrrr at google.com>
> wrote:
> > + if (!IsAligned(beg, SHADOW_GRANULARITY))
> > + Report("ERROR: beg is not aligned by %d\n", SHADOW_GRANULARITY);
>
> This has to be implemented in the same way as the original check:
>
> uptr granularity = SHADOW_GRANULARITY;
> Report("ERROR: beg is not aligned by %d\n", granularity);
>
> The reason being that macros are expanded differently in different
> architectures and the sizes, types and va_arg assumptions will break.
>
> Timur,
>
> I tested the change on my ARM machine and it prints the correct
> message, "aligned by 8". Feel free to reapply the patch with that
> modification.
>
> cheers,
> --renato
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150225/42f2195e/attachment.html>
More information about the llvm-commits
mailing list