[clang] [clang-format] Treat empty for/while loops as short loops (PR #70768)

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 31 14:31:23 PDT 2023


================
@@ -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:

Does it matter in the real world? Do we really need to decide whether the following is “short”?
```
/**/ while /**/ ( /**/ true /**/ ) /**/ ; //
```

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


More information about the cfe-commits mailing list