[PATCH] D68408: [InstCombine] Negator - sink sinkable negations

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D68408#1976039 <https://reviews.llvm.org/D68408#1976039>, @lebedev.ri wrote:

> 1. add `Abs` IR instruction (i think not)


Is the answer the same for an intrinsic? The more we see these kinds of problems, the more I wish we had created intrinsics for abs/smin/smax/umin/umax long ago. We keep trying to work around this limitation in IR, but it doesn't seem worth it. We have abs() functions in source and an abs node in codegen, so we're creating intermediate ops that don't exist on either side of IR.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68408