[PATCH] D137655: Expand fminimum and fmaximum into a pair of selects

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

I don't think this is right either. LLVM defines minnum according to the old semantics, which don't specify an order between zeroes. We'd need a separate ISD opcode for minnum according to 2018 semantics.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137655