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

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 14:18:35 PDT 2023


jrtc27 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.

As a separate commit though, please, so the diffs are clearer


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