[clang] [llvm] Minimal support of floating-point operand bundles (PR #135658)

[Serge Pavlov via llvm-commits]

================
@@ -4388,6 +4388,31 @@ void AssemblyWriter::printInfoComment(const Value &V) {
 
   if (PrintInstAddrs)
     Out << " ; " << &V;
+
+  if (auto *CI = dyn_cast<CallInst>(&V))
+    if (Intrinsic::ID IID = CI->getIntrinsicID())
+      if (IntrinsicInst::isFloatingPointOperation(IID))
+        if (const BasicBlock *BB = CI->getParent())
+          if (const Function *F = BB->getParent())
+            if (F->hasFnAttribute(Attribute::StrictFP)) {
----------------
spavloff wrote:

No, this important property of the proposed solution. Whether an intrinsic call has side effect is determined by the function that contains it. If the function runs in default mode, it cannot contain FP operations with side effects. And in in strictfp function all such operations have side effects.

https://github.com/llvm/llvm-project/pull/135658