[PATCH] D43142: Experimental pass to convert all floating point operations to the equivalent constrained intrinsics

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

kpn added a comment.

Isn't this pass needed since we don't have a way to prevent code movement between blocks using the pragma FENV_ACCESS ON and blocks that aren't? I suggest having it check for use of constrained intrinsics before making any changes.

And if it was a function pass we could rerun it after certain optimizations like, say, the inliner.

Thoughts? Anyone?


https://reviews.llvm.org/D43142