[PATCH] D53157: Teach the IRBuilder about constrained fadd and friends
Kevin P. Neal via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 3 10:51:52 PST 2018
kpn updated this revision to Diff 176432.
kpn added a comment.
I've changed the patch so that calls to CreateFAdd() et al will give you constrained intrinsics if they are enabled. This required adding functions to enable/disable constrained-as-default plus calls to deal with the rounding mode and exception handling for contrained intrinsics.
I've left the independent create constrained functions to make it dead easy to move them out of the header and avoid inlining bloat.
I'm not dealing with any fast math flags or metadata currently. I'm not sure they make sense in a constrained context, or at least I haven't seen anyone argue they must be in place for an initial implementation.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53157/new/
https://reviews.llvm.org/D53157
Files:
include/llvm/IR/IRBuilder.h
unittests/IR/IRBuilderTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53157.176432.patch
Type: text/x-patch
Size: 18309 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181203/6f93f13f/attachment-0001.bin>
More information about the cfe-commits
mailing list