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

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

> Just because FENV_ACCESS can be toggled on that granularity doesn't mean we have to represent it that way. We've previously agreed (I think) that if FENV_ACCESS is enabled anywhere in a function we will want to use the constrained intrinsics for all FP operations in the function, not just the ones in the scope where it was specified.

Yes, this is also my understanding. We can't soundly mix the two in the same function because we can't prevent the code motion within the function.


https://reviews.llvm.org/D53157