================
@@ -357,6 +357,9 @@ class IRBuilderBase {
void setConstrainedFPCallAttr(CallBase *I) {
I->addFnAttr(Attribute::StrictFP);
+ MemoryEffects ME = MemoryEffects::inaccessibleMemOnly();
----------------
andykaylor wrote:
I think all of the constrained intrinsics currently use inaccessibleMemOnly,
but it might be a good idea to get this attribute from the intrinsic itself. If
we added an intrinsic for something like sincos, it would access argument
memory too.
We should consider adding separate memory effects for fp control and status
registers. Obviously that would be a separate change set.
https://github.com/llvm/llvm-project/pull/109798
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
