commit 43b944e3c14bc68cd49510d208944499ee71b8a3
author Chih-Hung Hsieh <chh@google.com> Tue May 22 21:37:49 2018 -0700
committer Chih-hung Hsieh <chh@google.com> Wed May 23 17:15:48 2018 +0000
tree fe0dd002a41dd2e7b0a1e797230e874b8a9782a5
parent 281d56bbc8799578dcd65e9a5e06acacef38b5d0

Work around b/24465209, do not use clang lld

See longer explanation in b/80093890.
Clang lld does not generate expected DT_REL and DT_RELA tags
with --hash-style=both and --pack-dyn-relocs=android.
I am not sure about the extent of b/24465209, so
I would rather not to use lld for these .so files for now.

Bug: 80093890
Bug: 24465209
Test: build with USE_CLANG_LLD=true and run dlext.compat_elf_hash_and_relocation_tables
Change-Id: I645dbe25c5b9975605e3af5e717ed36c276f14d4

liblog/Android.bp

diff --git a/liblog/Android.bp b/liblog/Android.bp
index 2d5a5db..1bd796a 100644
--- a/liblog/Android.bp
+++ b/liblog/Android.bp
@@ -84,6 +84,7 @@
         android_arm: {
             // TODO: This is to work around b/24465209. Remove after root cause is fixed
             ldflags: ["-Wl,--hash-style=both"],
+            use_clang_lld: false,
         },
         windows: {
             srcs: ["uio.c"],