<div class="gmail_quote">On Fri, May 25, 2012 at 3:16 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">
<div class="im">On Wed, May 23, 2012 at 4:07 PM, Evgeniy Stepanov<br>
<<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>> wrote:<br>
> Good point. Both patches landed, here is another one that adds a<br>
> driver check for -pie.<br>
<br>
</div>ping w/ a patch<br></blockquote><div><br></div><div>LGTM</div><div><br></div><div>The diagnostic says that '-pie' is required with ASan on Android, but nothing about during compiling vs. during linking? Do we blanket require this flag, or does it only apply to some steps? Might be nice to make the diagnostic explicit here.</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>
<br>
> On Wed, May 23, 2012 at 3:04 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
>> On Wed, May 23, 2012 at 3:58 AM, Evgeniy Stepanov<br>
>> <<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>> wrote:<br>
>>><br>
>>> 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>
>><br>
>><br>
>> I haven't looked at the patches, but it would be good to also teach the<br>
>> Clang driver to produce an error if the appropriate options are not attached<br>
>> to the compile and link commands but '-faddress-sanitizer' is.<br>
>><br>
>>><br>
>>><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>
>><br>
>><br>
</div></div></blockquote></div><br>