================
@@ -1418,6 +1427,12 @@ void CodeGenFunction::EmitForStmt(const ForStmt &S,
 
   if (CGM.shouldEmitConvergenceTokens())
     ConvergenceTokenStack.pop_back();
+
+  if (FinalBodyBB) {
+    // Key Instructions: We want the for closing brace to be step-able on to
+    // match existing behaviour.
----------------
OCHyams wrote:

> Does this mean that for brace-less for loops, we could theoretically step on 
> the same last-line twice if there's a simple assignment and then a branch?

There could be two is_stmts there for the same line, yep (I think we discussed 
before that interpreting that is up to the debugger).

Added to test for additional coverage.

Updated comment.

https://github.com/llvm/llvm-project/pull/134646
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to