[PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 17 14:17:31 PST 2018

Previous message: [PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS
Next message: [PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

craig.topper added a comment.

You’ll need to use an intrinsic to test the changes Simon asked for. The existing combines for X86 do their own split for AVX1.


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

https://reviews.llvm.org/D55787

Previous message: [PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS
Next message: [PATCH] D55787: [X86] Use UADDSAT/USUBSAT instead of ADDUS/SUBUS
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list