================
@@ -3117,9 +3117,16 @@ void UnwrappedLineParser::parseForOrWhileLoop(bool 
HasParens) {
       FormatTok->setFinalizedType(TT_ConditionLParen);
     parseParens();
   }
-  // Event control.
-  if (Style.isVerilog())
+
+  if (Style.isVerilog()) {
+    // Event control.
     parseVerilogSensitivityList();
+  } else if (Style.AllowShortLoopsOnASingleLine && FormatTok->is(tok::semi) &&
----------------
owenca wrote:

I didn't use `getPreviousNonComment()` because there are 3 more cases where we 
either can't or probably shouldn't merge the semicolon:

1.
```
while (1) //
  ;
```
2.
```
while (1) /**/
  ;
```
3.
```
while (1)
  /**/ ;
```
Like `while (1) /**/ ;`, case 3 probably doesn't matter in practice.

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

Reply via email to