[PATCH] D58144: [xray][tests][RHDTS] Add -lstdc++ after LLVM libs, resolving link error with RHDTS

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 12 13:54:10 PST 2019


hubert.reinterpretcast marked an inline comment as done.
hubert.reinterpretcast added inline comments.


================
Comment at: lib/xray/tests/CMakeLists.txt:83
 
+list(APPEND XRAY_UNITTEST_LINK_FLAGS -l${SANITIZER_CXX_ABI_LIBRARY})
+
----------------
tstellar wrote:
> Any idea why the xray tests need to manually specify the standard library like this instead of letting the compiler driver do it automatically?
It uses the C compiler invocation for the linking, perhaps to avoid picking up the default C++ run-time libraries.


Repository:
  rCRT Compiler Runtime

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58144/new/

https://reviews.llvm.org/D58144





More information about the llvm-commits mailing list