[clang] [llvm] Implement operand bundles for floating-point operations (PR #109798)

Andy Kaylor via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 24 16:23:44 PDT 2024


================
@@ -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


More information about the cfe-commits mailing list