[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