[PATCH] D53157: Teach the IRBuilder about constrained fadd and friends

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added a comment.

In https://reviews.llvm.org/D53157#1290044, @spatel wrote:

> Should the builder's FastMathFlags setting apply to these calls? Is it worth adding an optional FMF parameter to these now, so we don't end up with duplicated API like we have for the regular FP binops?


In the interest of having these new calls be like the non-strict ones, I think this does make some sense.

Would it be enough to model after the existing FP binops and take an Instruction argument to copy the flags from?


https://reviews.llvm.org/D53157