[PATCH] D121757: [clang-format] Take out common code for parsing blocks

Thu Mar 17 09:25:27 PDT 2022

owenpan added a comment.

In D121757#3389092 <https://reviews.llvm.org/D121757#3389092>, @sstwcw wrote:

> This patch is only intended to reduce the number of times the functionality gets implemented separately.  Any change in behavior would be unintended.  And we also use the `parseIndentedBlock` in Verilog stuff, so it's not just two places.  I will wait for the check-clang result to see what to do.

In that case, I can accept this patch if the review comments are addressed. We can add inserting braces to `switch` in a separate patch. Not sure what name we should use for the function though as it would not be parsing the body of loop statements anymore.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121757