[PATCH] D95289: [TargetLowering][RISCV] Don't turn (seteq/ne (sext_inreg X, VT), C1) -> (seteq/ne (zext_inreg X, VT), C1) if the sext_inreg is cheaper

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 13:54:17 PST 2021


craig.topper added a comment.

In D95289#2521096 <https://reviews.llvm.org/D95289#2521096>, @luismarques wrote:

> Makes sense to me.
>
> @craig.topper are you using the correct clang-format config? Remember to fix the minor formatting issue.

I omitted that clang-format change from my patch because its not really in code I touched. Does clang-format widen its scope to include an else if that I that follows the one I changed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95289



More information about the llvm-commits mailing list