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

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 16 20:14:36 PST 2018


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





More information about the cfe-commits mailing list