Skip to content
Snippets Groups Projects
Commit 9ecbd6eb authored by Pirama Arumuga Nainar's avatar Pirama Arumuga Nainar
Browse files

Reorder static libs to appease ld.gold linker

Bug: http://b/112277682
Bug: http://b/79419131

Lld, which is the default linker for the platform, is able to resolve
symbols across libraries irrespective of the order they appear in the
command line.  This was not the case for ld.gold and other linkers.
Unfortunately, there are some targets that still rely on ld.gold
(pgo-taimen-config1 in internal release branch being one of
them).

With http://aosp/721652, libbt-stack should be added before libbt-common
so MessageLoopThread::~MessageLoopThread is resolved correctly.  If not,
the following error is reported by gold:

system/bt/stack/btu/btu_init.cc:41: error: undefined reference to
'bluetooth::common::MessageLoopThread::MessageLoopThread(std::__1::basic_string<char,
std::__1::char_traits<char>, std::__1::allocator<char> > const&)'

Test: m USE_CLANG_LLD=false
Change-Id: I9c8208218ffb4bb027ad75b51bcd484e9bb37368
parent 24c5a2c6
No related branches found
No related tags found
No related merge requests found
......@@ -213,8 +213,8 @@ cc_test {
],
static_libs: [
"libbt-bta",
"libbt-common",
"libbt-stack",
"libbt-common",
"libbt-sbc-decoder",
"libbt-sbc-encoder",
"libFraunhoferAAC",
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment