[PATCH] D151954: [clang-format] Fix overlapping whitespace replacements before PPDirective

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 7 11:54:53 PDT 2023


owenpan added a comment.

In D151954#4403921 <https://reviews.llvm.org/D151954#4403921>, @paulkirth wrote:

> This patch seems to introduce some significant regressions to formatting. I've filed https://github.com/llvm/llvm-project/issues/63170 with some candidate code. I'll try to reduce the case as I can, but I'm guessing something is incorrect w/ how the indentation level is tracked, since we see includes being incorrectly indented and lots of lines w/ whitespace.

https://github.com/llvm/llvm-project/issues/63170#issuecomment-1581331587


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151954



More information about the cfe-commits mailing list