DescriptionMisc undefined symbol fixes for Android+LTO.
It appears that at least in some cases, the linker uses a stricter set of
rules when linking LTO objects, which causes the link to fail in some cases
where an unused function depends on an undefined symbol. This change fixes
all such cases where the error caused the LTO build to fail, by either
moving source files between lists, or by selectively disabling parts of
source files depending on the configuration.
There are more errors like this, which for some reason do not affect LTO. They
can be inspected like this (with a regular Chromium/android build):
$ ninja -C out/Release chrome_shell_apk
$ nm out/Release/lib/libchromeshell.so | grep " U _Z"
BUG=407544
R=thakis@chromium.org
Committed: https://crrev.com/3a725956aff76414de8ecec2dcaa773b21fc8fe5
Cr-Commit-Position: refs/heads/master@{#325325}
Patch Set 1 #
Total comments: 2
Patch Set 2 : Move #include block down #
Messages
Total messages: 8 (2 generated)
|