[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