[PATCH] D71218: [FPEnv] Remove rounding mode argument for constrained minnum/maxnum

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 11:11:43 PST 2019


uweigand created this revision.
uweigand added reviewers: kpn, andrew.w.kaylor, craig.topper, cameron.mcinally, RKSimon.
Herald added subscribers: llvm-commits, jdoerfert, jsji, kbarton, nemanjai.
Herald added a project: LLVM.

The llvm.experimental.constrained.minnum and llvm.experimental.constrained.maxnum intrinsics currenly carry a rounding mode metadata argument.  This is not useful since the semantics of minnum and maxnum do not in any way depend on the rounding mode.  In similar cases, the constrained intrinsics do not have the rounding mode argument.  Remove it here as well.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D71218

Files:
  llvm/docs/LangRef.rst
  llvm/include/llvm/IR/ConstrainedOps.def
  llvm/include/llvm/IR/Intrinsics.td
  llvm/test/CodeGen/ARM/fp-intrinsics.ll
  llvm/test/CodeGen/PowerPC/ppcf128-constrained-fp-intrinsics.ll
  llvm/test/CodeGen/PowerPC/vector-constrained-fp-intrinsics.ll
  llvm/test/CodeGen/SystemZ/vec-strict-max-01.ll
  llvm/test/CodeGen/SystemZ/vec-strict-min-01.ll
  llvm/test/CodeGen/SystemZ/vector-constrained-fp-intrinsics.ll
  llvm/test/CodeGen/X86/fp128-libcalls-strict.ll
  llvm/test/CodeGen/X86/vector-constrained-fp-intrinsics.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71218.232900.patch
Type: text/x-patch
Size: 43510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191209/d62a2502/attachment.bin>


More information about the llvm-commits mailing list