[PATCH] D55780: [X86] Create PSUBUS from (add (umax X, C), -C)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 18 06:15:40 PST 2018


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

This change handles psubus cases with no undefs, so LGTM. 
Depending on how the patches land, we could add the undef capability and tests for it directly here or make that an enhancement.
But as I mentioned, I don't think this patch alone is enough to close PR40053 because that's at least 2 independent bugs in 1 report.


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

https://reviews.llvm.org/D55780





More information about the llvm-commits mailing list