[PATCH] D147173: [RISCV] Merge SExtWRemoval and StripWSuffix into a single pass.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 15:00:06 PDT 2023


craig.topper added a comment.

In D147173#4231929 <https://reviews.llvm.org/D147173#4231929>, @asb wrote:

> LGTM. I'd note that clang-format prefers different formatting for some of the functions here - it's probably worth reformatting while you're doing this merge into a new pass.

Is it just the switch statement I intentionally wrote with the lines compacted?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147173



More information about the llvm-commits mailing list