<div dir="ltr">Macros are hard :(<br><br>Relanded as r230506, thanks for the help!</div><br><div class="gmail_quote">On Tue Feb 24 2015 at 11:21:31 PM Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I found the problem...<br>
<br>
On 20 February 2015 at 15:45, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>> wrote:<br>
> +  if (!IsAligned(beg, SHADOW_GRANULARITY))<br>
> +    Report("ERROR: beg is not aligned by %d\n", SHADOW_GRANULARITY);<br>
<br>
This has to be implemented in the same way as the original check:<br>
<br>
    uptr granularity = SHADOW_GRANULARITY;<br>
    Report("ERROR: beg is not aligned by %d\n", granularity);<br>
<br>
The reason being that macros are expanded differently in different<br>
architectures and the sizes, types and va_arg assumptions will break.<br>
<br>
Timur,<br>
<br>
I tested the change on my ARM machine and it prints the correct<br>
message, "aligned by 8". Feel free to reapply the patch with that<br>
modification.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div>