[PATCH] D121714: [VP] fm flag transfer to SDNodes [VE] VVP_FMA fusion

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:7530
+      NodeFlags.setNoFPExcept(true); // TODO: Constrained VP
+      Result->setFlags(NodeFlags);
+    }
----------------
Is this correct if the getNode call CSEs with an existing node? Should probably pass the flags to getNode.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121714/new/

https://reviews.llvm.org/D121714