[Lldb-commits] [lldb] [lldb][unittests] Add LLDB_UNITTEST_STRIP_DEBUG_INFO to cut unittest link time (PR #203274)

[Charles Zablit via lldb-commits]

charles-zablit wrote:

> I'm not sure about this. But since it's off by default, it might be fine. One reason for the slow link times is probably the fact that LLVM and Clang are statically linked on Windows.

Yes, the resulting PDBs for the tests is close to 2GB for each binary. I am not sure how to express this as a CMakeUserPresets.json to not force it into the codebase. It's really just for the developer's convenience as I've noticed that the link time is in the order of several minutes for the unittests.

As you said, it should be fine since it's OFF by default.

> `/OPT:REF` and `/OPT:ICF` are mostly for binary size, not for linking speed.

This might be overkill. I'm happy to remove it.

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