[PATCH] D96413: [DAG] Move basic USUBSAT pattern matches from X86 to DAGCombine

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 11:49:31 PST 2021


nikic added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/usub_sat.ll:35
+; CHECK-NEXT:    subs w8, w8, w1, uxth
+; CHECK-NEXT:    csel w0, wzr, w8, lo
 ; CHECK-NEXT:    ret
----------------
Why are tests that directly use usub.sat intrinsics affected by this? Are we doing something weird like first expanding them to min/max sub and then combining them back to usubsat on the extended type?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96413



More information about the llvm-commits mailing list