[PATCH] D36335: Add ‘llvm.experimental.constrained.fma‘ Intrinsic

[Wei Ding via Phabricator via llvm-commits]

wdng added inline comments.


================
Comment at: lib/Target/X86/X86ISelDAGToDAG.cpp:2015
   default: break;
+  case ISD::FMA: {
+    SDValue ISDFMA = CurDAG->getNode(X86ISD::FMADD, SDLoc(Node),
----------------
andrew.w.kaylor wrote:
> Can you explain why this was necessary?  I would have expected there to have been handling already in place for ISD::FMA.
No it doesn't, looks like X86 doesn't handle ISD:FMA automatically unless we there is -mattr=+fma option. Without this, CodeGen/X86/fp-intrinsics.ll will fail in instruction selection.


Repository:
  rL LLVM

https://reviews.llvm.org/D36335