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

[Wei Ding via Phabricator via llvm-commits]

wdng added a comment.

In https://reviews.llvm.org/D36335#832367, @b-sumner wrote:

> An update to docs/LangRef.rst is needed.


Sure, will do.



================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:6651
   case ISD::STRICT_FREM: NewOpc = ISD::FREM; break;
+  case ISD::STRICT_FMA: NewOpc = ISD::FMA; break;
   case ISD::STRICT_FSQRT: NewOpc = ISD::FSQRT; IsUnary = true; break;
----------------
b-sumner wrote:
> This is a ternary operation.  Code below assumes unary or binary.
In Intrinsics.td, we have defined fma is a ternary operator. Here it only mutates STRICT_FMA to FMA and IsUnary is false by default. So we may not need to specify whether is unary of binary here?


Repository:
  rL LLVM

https://reviews.llvm.org/D36335