[PATCH] D122051: [RISCV] The immediate version of sgt/ugt lowering to slti/sltiu + xori

[Ben Shi via Phabricator via llvm-commits]

benshi001 added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/fpclamptosat_vec.ll:1484
 ; CHECK-NEXT:  .LBB18_8: # %entry
-; CHECK-NEXT:    slt a1, a0, a1
 ; CHECK-NEXT:    bne s1, a0, .LBB18_12
----------------
This case becames poor, due to a0 (containing -1) has mutlple uses. So I think your need an extra check, that the immediate (`simm12_sub1_exc0`) has only one use. 


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

https://reviews.llvm.org/D122051