================
@@ -1749,6 +1757,8 @@ void CodeGenFunction::EmitCaseStmtRange(const CaseStmt &S,
   // switch machinery to enter this block.
   llvm::BasicBlock *CaseDest = createBasicBlock("sw.bb");
   EmitBlockWithFallThrough(CaseDest, &S);
+  if (getLangOpts().CPlusPlus)
----------------
efriedma-quic wrote:

I agree that it's probably fine if the initial patch misses initializing some 
variables in functions with computed goto, but I don't think it's okay to 
miscompile by overwriting variables that should not be reinitialized.

With that in mind, maybe we can simplify a bit, but we can't ignore the 
semantics.

https://github.com/llvm/llvm-project/pull/181937
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to