[clang] [clang] Merge gtest binaries into AllClangUnitTests (PR #134196)

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Mon May 12 09:09:20 PDT 2025


================
@@ -304,7 +304,7 @@ getCodeModel(const CodeGenOptions &CodeGenOpts) {
                            .Case("kernel", llvm::CodeModel::Kernel)
                            .Case("medium", llvm::CodeModel::Medium)
                            .Case("large", llvm::CodeModel::Large)
-                           .Case("default", ~1u)
+                           .Cases("default", "", ~1u)
----------------
rnk wrote:

I responded on Discord, but just to resolve the comment here on github:
I recall there was a case where the main LLVM target gets registered, but the MCTargetDesc isn't registered, and this resulted in some failure along some codepath. Of course, if you can remove it and tests pass, anyone is welcome to revisit that. It's not obvious why this is needed.

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


More information about the cfe-commits mailing list