[Lldb-commits] [lldb] [lldb][test] Enable static linking with libcxx for import-std-module tests (PR #98701)

Michael Buch via lldb-commits lldb-commits at lists.llvm.org
Wed Oct 16 02:21:56 PDT 2024


Michael137 wrote:

> I guess it will with clang+lld (which is the only way we can run the test suite now), though I still find this suboptimal -- in particular, I think the multiple definition thingy plants us firmly into undefined behavior territory.
> 
> Did you check that `at(0)` actually causes the necessary symbols to be pulled into the binary? If yes, why can't we find them? If not, why not?

Agree with Pavel here, it would be good to figure out what exactly is going wrong in this test.

FWIW, we don't actually want to emit `at` into the debug-info. This test specifically tests the case where we don't have the debug-info for a function, but are able to find and call it using the `std` Clang module. IIRC from the last time we saw this, the problem only occurs when debug symbols for libc++ are available (https://reviews.llvm.org/D139361).

Since this is quite tricky to debug, and has been a known issue for a while and (i think) only happens when we built libc++ with debug symbols, I'd be fine with skipping the test on Linux until we figure out what the root cause is.

https://github.com/llvm/llvm-project/pull/98701


More information about the lldb-commits mailing list