[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