[PATCH] D59006: [x86] improve the default expansion of uaddsat/usubsat

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri accepted this revision.
lebedev.ri added a comment.

This LG, but i'm not sure i understand how this is related to D59066 <https://reviews.llvm.org/D59066>?
Here, we are clearly end up with no `select` in ASM.
But in D59066 <https://reviews.llvm.org/D59066> we expand to this pattern.
So there is something else that is able to do the transform that we do manually in D59066 <https://reviews.llvm.org/D59066>?
Should D59066 <https://reviews.llvm.org/D59066> be doing something else to simply trigger the existing transform?


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

https://reviews.llvm.org/D59006