[PATCH] D46967: Vector constrained FP intrinsics

[Andy Kaylor via Phabricator via llvm-commits]

andrew.w.kaylor added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp:312
+    // ISD::STRICT_FSQRT.
+    Node = DAG.mutateStrictFPToFP(Node);
+    QueryType = Node->getValueType(0);
----------------
Do you actually want to mutate the node at this point?

There's a function, getStrictFPOpcodeAction() that does something very close to what you are doing here except without mutating the node. Would moving that to an accessible location do what you need?


Repository:
  rL LLVM

https://reviews.llvm.org/D46967