================
@@ -357,6 +357,9 @@ class IRBuilderBase {
 
   void setConstrainedFPCallAttr(CallBase *I) {
     I->addFnAttr(Attribute::StrictFP);
+    MemoryEffects ME = MemoryEffects::inaccessibleMemOnly();
----------------
jcranmer-intel wrote:

It should be possible to make `CallBase::getMemoryEffects` return the 
appropriate effects for the bundle operand, instead of needing to set it 
correctly.

(as a side note, since this is definitely follow-up patch territory, it 
probably makes sense to eventually add a FP environment mem effect separate 
from inaccessible mem only).

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

Reply via email to