diff options
author | David Srbecky <dsrbecky@google.com> | 2018-03-13 11:31:22 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2018-03-13 11:31:22 +0000 |
commit | ff9b1db67c930a3e04ed29714d8b872e92e27b85 (patch) | |
tree | 96926fbb0ebba62c9a15608cad51ba8614dac334 | |
parent | 9f5e6dbe854ca41d337a1fa8fc8efe95e22db04b (diff) | |
parent | 18121dc4d04c0444eff3b7a4e1460f07cbc4fcc8 (diff) | |
download | system_core-ff9b1db67c930a3e04ed29714d8b872e92e27b85.tar.gz system_core-ff9b1db67c930a3e04ed29714d8b872e92e27b85.tar.bz2 system_core-ff9b1db67c930a3e04ed29714d8b872e92e27b85.zip |
Merge "Remove libunwindstack_dex"
-rw-r--r-- | libunwindstack/Android.bp | 83 |
1 files changed, 9 insertions, 74 deletions
diff --git a/libunwindstack/Android.bp b/libunwindstack/Android.bp index 124c70e83..87172836b 100644 --- a/libunwindstack/Android.bp +++ b/libunwindstack/Android.bp @@ -47,6 +47,8 @@ cc_library { srcs: [ "ArmExidx.cpp", + "DexFile.cpp", + "DexFiles.cpp", "DwarfCfa.cpp", "DwarfEhFrameWithHdr.cpp", "DwarfMemory.cpp", @@ -85,11 +87,13 @@ cc_library { }, vendor: { cflags: ["-DNO_LIBDEXFILE_SUPPORT"], - exclude_static_libs: ["libunwindstack_dex"], + exclude_srcs: [ + "DexFile.cpp", + "DexFiles.cpp", + ], exclude_shared_libs: ["libdexfile"], }, }, - whole_static_libs: ["libunwindstack_dex"], arch: { x86: { @@ -114,79 +118,9 @@ cc_library { ], } -// Isolate the dex file processing into a separate library. Currently, -// it is necessary to add art include directories directly, which also -// adds the art elf.h file in the include path, overriding the system one. -// Work to isolate libdexfile is b/72216369. -cc_library_static { - name: "libunwindstack_dex", - vendor_available: false, - defaults: ["libunwindstack_flags"], - - cflags: [ - "-Wexit-time-destructors", - ], - - srcs: [ - "DexFile.cpp", - "DexFiles.cpp", - ], - target: { - // Always disable optimizations for host to make it easier to debug. - host: { - cflags: [ - "-O0", - "-g", - ], - }, - }, - - shared_libs: [ - "libbase", - "libdexfile", - ], - local_include_dirs: ["include"], - allow_undefined_symbols: true, - - // libdexfile will eventually properly export headers, for now include - // these directly. - include_dirs: [ - "art/runtime", - ], -} - //------------------------------------------------------------------------- // Unit Tests //------------------------------------------------------------------------- -cc_test_library { - name: "libunwindstack_dex_test", - vendor_available: false, - defaults: ["libunwindstack_flags"], - - shared: { - enabled: false, - }, - - srcs: [ - "tests/DexFileTest.cpp", - "tests/DexFilesTest.cpp", - ], - local_include_dirs: ["include"], - allow_undefined_symbols: true, - - shared_libs: [ - "libbase", - "libunwindstack", - "libdexfile", - ], - - // libdexfile will eventually properly export headers, for now include - // these directly. - include_dirs: [ - "art/runtime", - ], -} - cc_test { name: "libunwindstack_test", defaults: ["libunwindstack_flags"], @@ -194,6 +128,8 @@ cc_test { srcs: [ "tests/ArmExidxDecodeTest.cpp", "tests/ArmExidxExtractTest.cpp", + "tests/DexFileTest.cpp", + "tests/DexFilesTest.cpp", "tests/DwarfCfaLogTest.cpp", "tests/DwarfCfaTest.cpp", "tests/DwarfDebugFrameTest.cpp", @@ -242,14 +178,13 @@ cc_test { "liblog", "liblzma", "libunwindstack", + "libdexfile", ], static_libs: [ "libgmock", ], - whole_static_libs: ["libunwindstack_dex_test"], - data: [ "tests/files/elf32.xz", "tests/files/elf64.xz", |