[PATCH] D15308: [tsan] Support aarch64 for Android.

[Adhemerval Zanella via llvm-commits]

zatrazz added inline comments.

================
Comment at: lib/tsan/rtl/tsan_platform.h:129
@@ +128,3 @@
+
+#else
+
----------------
yabinc wrote:
> eugenis wrote:
> > yabinc wrote:
> > > eugenis wrote:
> > > > Is there a reason you are not using the generic aarch64/linux mappings below?
> > > > For ASan/aarch64/android, I've noticed that ART requires a first 4GB mapping, which seems incompatible with this layout.
> > > > 
> > > The Android mapping is different from Mapping39 in kLoAppMemBeg-kLoAppMemEnd. Because
> > > android links all executables as shared libraries.
> > > Thanks for reminding. I didn't notice that. I will check it out after my vacation.
> > You mean PIE?
> > That linux mapping does not support PIE!
> > Well it could be a kernel version thing - at some point they moved PIE executable mappings from 0x7f.. to 0x55...
> > 
> > I've added a comment to http://reviews.llvm.org/D14199.
> > 
> > I don't mind going ahead with this one and figuring out if we want to unify the mappings later.
> > 
> Yes, I mean PIE. It might be related to kernel version, or architecture. I am not sure.
As I commented on the aarch64-linux TSAN mapping unification discussion [1] I did check the tests with -fpic -pie build and I saw no issue. I checked on a recent kernel (3.19) and on this configuration the PIE addresses for TSAN builds are se on 0x7f.. range.

I will check for 42-bits if TSAN tests fail with PIE build.

[1] http://reviews.llvm.org/D14199


http://reviews.llvm.org/D15308