[PATCH] D154093: [clang-format] Break long strings in Verilog

[Björn Schäpers via Phabricator via cfe-commits]

HazardyKnusperkeks added a comment.

Apart from that I'm okay.



================
Comment at: clang/lib/Format/WhitespaceManager.cpp:1428
+      // case, it is allowed if there is a replacement for the empty range
+      // between 2 tokens and another non-empty range at the start of the second
+      // token.
----------------
HazardyKnusperkeks wrote:
> Can you elaborate where the 2 replacements come from, and why we can't make sure there is no empty range?
This is open (at least for me).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154093