[PATCH] D151358: [LegalizeTypes] Improve expansion of wide SMIN/SMAX/UMIN/UMAX

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 24 12:01:34 PDT 2023


efriedma added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/min.ll:568
 
-; EG: MIN_UINT
-; EG: MIN_UINT
+; EG: SETE_INT
+; EG: SETGT_UINT
----------------
arsenm wrote:
> I'm guessing you need to check if the wide setcc is legal/good 
Determining "good" is tricky... RISCV doesn't have any special instructions for wide setcc, but this change is still profitable there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151358



More information about the llvm-commits mailing list