[PATCH] D158053: [PowerPC] Implement llvm.maximum/minimum intrinsic

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

NaN handling is not the only difference between these intrinsics, they also have different signed zero semantics.

Also, I really don't think another target should implement custom lowering for this without adding generic legalization support first. Please add generic legalization first, and if that's not optimal for ppc, add custom lowering. It makes no sense that every target is re-implementing this using custom lowering right now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158053