[PATCH] D62158: [InstCombine] canonicalize minnum/maxnum with 'nnan' to fcmp+select

Fri Jun 21 11:36:42 PDT 2019

spatel abandoned this revision.
spatel added a comment.

Sorry - I should have abandoned this. There's some patch spaghetti here.

I think we will go with the more conventional approach of 'less-instructions-is-more-canonical':
D62414 <https://reviews.llvm.org/D62414> - so the opposite direction of this.

That patch could get committed now because it doesn't have a direct dependency on anything, but I was hoping for a re-ruling on D63214 <https://reviews.llvm.org/D63214> which was itself dependent on the now abandoned D63294 <https://reviews.llvm.org/D63294>.

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

https://reviews.llvm.org/D62158