[PATCH] D143825: [clang-format] Put ports on separate lines in Verilog module headers

sstwcw via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 19 14:40:28 PST 2023


sstwcw added inline comments.


================
Comment at: clang/lib/Format/TokenAnnotator.cpp:2665
+    if (Style.isVerilog() && Precedence == prec::Comma &&
+        VerilogFirstOfType != nullptr) {
+      addFakeParenthesis(VerilogFirstOfType, prec::Comma);
----------------
owenpan wrote:
> And other places as well.
@HazardyKnusperkeks Do you agree?  I remember you once said you preferred the `nullptr` style.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143825/new/

https://reviews.llvm.org/D143825



More information about the cfe-commits mailing list