================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to