[clang] [clang] [unittest] Add a test for Generic_GCC::GCCVersion::Parse (PR #69078)

Martin Storsjö via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 19 22:33:47 PDT 2023


mstorsjo wrote:

> I hope that we do not drop `LLVM_LIBRARY_VISIBILITY` arbitrarily from `clang::driver::toolchains::*` classes, just because some unittests need to reference the symbols in a shared object.

That’s a reasonable point.

> ```c
> #if !defined(LLVM_BUILD_SHARED_LIBS)
> ```
> 
> is not great but is not too bad. `-DBUILD_SHARED_LIBS=on` modes are slow to execute tests and are not used often for Release builds.

I guess that’s a reasonable tradeoff. We’d need to do the same for dylib mode as well, which probably is used a bit more. But the main point is that whoever is working on modifying that implementation can run the tests fairly easy, and that they get run somewhere in some configurations at least.

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


More information about the cfe-commits mailing list