[PATCH] D62026: [Intrinsics] Merge lround.i32 and lround.i64 into a single intrinsic with overloaded result type. Make result type for llround overloaded instead of fixing to i64

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 12:25:43 PDT 2019


craig.topper created this revision.
craig.topper added reviewers: zatrazz, efriedma, SjoerdMeijer.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.

We shouldn't really make assumptions about possible sizes for long and long long. And longer term we should probably support vectorizing these intrinsics. By making the result types not fixed we can support vectors as well.


https://reviews.llvm.org/D62026

Files:
  clang/lib/CodeGen/CGBuiltin.cpp
  clang/test/CodeGen/math-builtins.c
  clang/test/CodeGen/math-libcalls.c
  llvm/docs/LangRef.rst
  llvm/include/llvm/IR/Intrinsics.td
  llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  llvm/lib/IR/Verifier.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62026.199881.patch
Type: text/x-patch
Size: 7486 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190516/18d82550/attachment.bin>


More information about the llvm-commits mailing list