On Fri, May 25, 2012 at 3:16 AM, Evgeniy Stepanov <[email protected] > wrote:
> On Wed, May 23, 2012 at 4:07 PM, Evgeniy Stepanov > <[email protected]> wrote: > > Good point. Both patches landed, here is another one that adds a > > driver check for -pie. > > ping w/ a patch > LGTM 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. > > > > On Wed, May 23, 2012 at 3:04 PM, Chandler Carruth <[email protected]> > wrote: > >> On Wed, May 23, 2012 at 3:58 AM, Evgeniy Stepanov > >> <[email protected]> wrote: > >>> > >>> Note that this change will require building binaries with -fPIE -pie. > >>> I believe there is really no way around this. We can not change shadow > >>> location on a per-binary basic because of the shared runtime, and > >>> there is no universal location that fit both PIE and non-PIE. > >>> > >>> I'm going to land this soon, then add some runtime checks to ensure > >>> that the shadow does not silently override any preexisting mappings. > >> > >> > >> 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. > >> > >>> > >>> > >>> On Tue, May 22, 2012 at 6:43 PM, Kostya Serebryany <[email protected]> > wrote: > >>> > Both look good, thanks! > >>> > --kcc > >>> > > >>> > On Tue, May 22, 2012 at 6:38 PM, Evgeniy Stepanov > >>> > <[email protected]> wrote: > >>> >> > >>> >> Hi, > >>> >> > >>> >> these patches (one to llvm, another to compiler-rt) change ASan > >>> >> default shadow offset on Android to zero. This seems to marginally > >>> >> improve performance of Thumb code, significantly (~5%) saves code > >>> >> size, and avoids address space collisions with the main executable > in > >>> >> a non-ASLR case. > >>> >> > >>> >> Please review. > >>> >> > >>> >> _______________________________________________ > >>> >> llvm-commits mailing list > >>> >> [email protected] > >>> >> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits > >>> >> > >>> > > >>> _______________________________________________ > >>> llvm-commits mailing list > >>> [email protected] > >>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits > >> > >> >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
