<div class="gmail_quote">On Wed, May 23, 2012 at 3:58 AM, Evgeniy Stepanov <span dir="ltr"><<a href="mailto:eugeni.stepanov@gmail.com" target="_blank">eugeni.stepanov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Note that this change will require building binaries with -fPIE -pie.<br>
I believe there is really no way around this. We can not change shadow<br>
location on a per-binary basic because of the shared runtime, and<br>
there is no universal location that fit both PIE and non-PIE.<br>
<br>
I'm going to land this soon, then add some runtime checks to ensure<br>
that the shadow does not silently override any preexisting mappings.<br></blockquote><div><br></div><div>I haven't looked at the patches, but it would be good to also teach the Clang driver to produce an error if the appropriate options are not attached to the compile and link commands but '-faddress-sanitizer' is.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On Tue, May 22, 2012 at 6:43 PM, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>
> Both look good, thanks!<br>
> --kcc<br>
><br>
> On Tue, May 22, 2012 at 6:38 PM, Evgeniy Stepanov<br>
> <<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> these patches (one to llvm, another to compiler-rt) change ASan<br>
>> default shadow offset on Android to zero. This seems to marginally<br>
>> improve performance of Thumb code, significantly (~5%) saves code<br>
>> size, and avoids address space collisions with the main executable in<br>
>> a non-ASLR case.<br>
>><br>
>> Please review.<br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>><br>
><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br>