New issue
Advanced search Search tips

Issue 807969 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

ToTAndroidCFI fails to link with "IDs have conflicting values"

Project Member Reported by h...@chromium.org, Feb 1 2018

Issue description

From https://ci.chromium.org/buildbot/chromium.clang/ToTAndroidCFI/1014:

[54315/56943] SOLINK ./lib_components_unittests__library.so
FAILED: lib_components_unittests__library.so lib_components_unittests__library.so.TOC lib.unstripped/lib_components_unittests__library.so 
python "../../build/toolchain/gcc_solink_wrapper.py" --readelf="../../third_party/android_ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/bin/arm-linux-androideabi-readelf" --nm="../../third_party/android_ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/bin/arm-linux-androideabi-nm" --strip=../../third_party/eu-strip/bin/eu-strip --sofile="./lib.unstripped/lib_components_unittests__library.so" --tocfile="./lib_components_unittests__library.so.TOC" --output="./lib_components_unittests__library.so"  -- ../../third_party/llvm-build/Release+Asserts/bin/clang++ -shared -fPIC -Wl,-z,noexecstack -Wl,-z,now -Wl,-z,relro --gcc-toolchain=../../third_party/android_ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64 -fuse-ld=lld -flto=thin -Wl,--thinlto-jobs=8 -Wl,--thinlto-cache-dir=thinlto-cache -Wl,--thinlto-cache-policy,cache_size=10\% -Wl,--lto-O0 -fwhole-program-vtables -Wl,--no-undefined -Wl,--exclude-libs=libgcc.a -Wl,--exclude-libs=libc++_static.a -Wl,--exclude-libs=libvpx_assembly_arm.a --target=arm-linux-androideabi -Werror -Wl,--warn-shared-textrel -Wl,-O1 -Wl,--gc-sections -Wl,--fatal-warnings -nostdlib -Wl,--warn-shared-textrel --sysroot=../../third_party/android_ndk/platforms/android-16/arch-arm -fsanitize=cfi-vcall -fsanitize=cfi-derived-cast -fsanitize=cfi-unrelated-cast -fno-sanitize-trap=cfi -Wl,--version-script=/b/c/b/ToTAndroidCFI/src/build/android/android_only_jni_exports.lst -Wl,-wrap,calloc -Wl,-wrap,free -Wl,-wrap,malloc -Wl,-wrap,memalign -Wl,-wrap,posix_memalign -Wl,-wrap,pvalloc -Wl,-wrap,realloc -Wl,-wrap,valloc -Wl,--dynamic-linker,/system/bin/linker -L../../third_party/android_ndk/sources/cxx-stl/llvm-libc++/libs/armeabi-v7a -o "./lib.unstripped/lib_components_unittests__library.so" -Wl,-soname="lib_components_unittests__library.so" @"./lib_components_unittests__library.so.rsp"
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
/b/c/b/ToTAndroidCFI/src/out/Release/../../third_party/llvm-build/Release+Asserts/bin/ld.lld: error: linking module flags 'Dwarf Version': IDs have conflicting values
clang-7.0: error: linker command failed with exit code 1 (use -v to see invocation)


I couldn't find any green builds from this bot, so I assume it's not a clang blocker.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 7 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/602e33bcf0b6e9a69130ab906000cd7ff4a2e0fc

commit 602e33bcf0b6e9a69130ab906000cd7ff4a2e0fc
Author: Peter Collingbourne <pcc@chromium.org>
Date: Wed Feb 07 09:58:59 2018

v8binding: Add core_config_add to configuration set for testing target.

This appears to be an oversight. It was exposed by r532545 which caused
default_symbols to be added to core_config_remove unconditionally. This
caused the testing target to be missing a symbols config, which
caused its translation units to be missing a DWARF version flag,
which resulted in DWARF version mismatch errors at link time in ARM
LTO sanitizer builds.

Bug:  807969 
Change-Id: I306a0e5d5f845b7d73be5ca10bb1c1c378ffd528
Reviewed-on: https://chromium-review.googlesource.com/905393
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534970}
[modify] https://crrev.com/602e33bcf0b6e9a69130ab906000cd7ff4a2e0fc/third_party/WebKit/Source/bindings/core/v8/BUILD.gn

Comment 2 by p...@chromium.org, Feb 9 2018

Status: Verified (was: Assigned)

Sign in to add a comment