It shouldn't be assumed that libstdc++ is always available even on Linux.
Just let the compiler pick the default
Details
- Reviewers
phosek delcypher vitalybuka - Commits
- rG4df44a008472: [asan] Don't use libstdc++ in test
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
compiler-rt/test/asan/TestCases/throw_invoke_test.cpp | ||
---|---|---|
2 | Why do we need static-libstdc? |
compiler-rt/test/asan/TestCases/throw_invoke_test.cpp | ||
---|---|---|
2 | I've just left it because that is what %linux_static_libstdcplusplus used to expand to. Though clearly it should be fine without because non Linux platforms omit this flag. Do you reckon we should remove it entirely? |
compiler-rt/test/asan/TestCases/throw_invoke_test.cpp | ||
---|---|---|
2 | this test runs on windows and osx, so you need some Linux specific substitution |
compiler-rt/test/lit.common.cfg.py | ||
---|---|---|
653 | -static-libstdc++ despite the name isn't specific to libstdc++, it applies to whichever -stdlib= is selected. I think it should be sufficient to just drop -stdlib=libstdc++. |
compiler-rt/test/lit.common.cfg.py | ||
---|---|---|
653 | Right, that is why I have just replaced the use of linux_static_libstdcplusplus with just -static-libstdc++ in the one place it was being used. On the other platforms that wasn't being used, but I doubt that it would make the test wrong on those platforms |
Why do we need static-libstdc?