[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