Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0f975a20 authored by Chih-Hung Hsieh's avatar Chih-Hung Hsieh Committed by Chih-hung Hsieh
Browse files

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
parent 931a840e

liblog/Android.bp

+1 −0
Original line number Original line Diff line number Diff line
@@ -84,6 +84,7 @@ cc_library { 
        android_arm: {

        android_arm: {

            // TODO: This is to work around b/24465209. Remove after root cause is fixed

            // TODO: This is to work around b/24465209. Remove after root cause is fixed

            ldflags: ["-Wl,--hash-style=both"],

            ldflags: ["-Wl,--hash-style=both"],

            use_clang_lld: false,

        },

        },

        windows: {

        windows: {

            srcs: ["uio.c"],

            srcs: ["uio.c"],