[PATCH] [clang] Use -android environment for all compiler-rt libs.
Evgeniy Stepanov
eugenis at google.com
Tue Jan 27 03:54:05 PST 2015
I suggest moving this logic into getCompilerRT and getting rid of the Env argument.
Beside the two functions this patch is changing, getCompilerRT is used in some MSVC-specific code which can not have llvm::Triple::Android, and it addSanitizerDynamicList.
The latter is interesting, because the current code is wrong, and the proposed change would fix it. Dot-sym files for android static runtime libraries are already generated with "-android" suffix, and the current implementation would fail to find them. There is no way to test it because android target defaults to shared runtime and there is no -static-libasan flag (as a counterpart to -shared-libasan).
REPOSITORY
rL LLVM
http://reviews.llvm.org/D7187
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list