[PATCH] D105853: [ISel] Expand saddsat and ssubsat via asr and xor

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 12 15:56:26 PDT 2021


dmgreen added inline comments.


================
Comment at: llvm/test/CodeGen/X86/ssub_sat.ll:33
+; X64-NEXT:    subl %esi, %eax
+; X64-NEXT:    cmovol %ecx, %eax
 ; X64-NEXT:    retq
----------------
Now that I look again, some of the individual X86 tests have increased, although the total size of the files has gone down.

What do we usually do about such things? I could just custom lower this in ARM/AArch64, but it appears to be better in general.


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

https://reviews.llvm.org/D105853



More information about the llvm-commits mailing list