Hi Evan,<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Right, the comment should not talk about "legal". The code in SelectionDAGBuilder.cpp should check TLI.isOperationLegal(ISD::FMA, ty) in addition to isFMACheap(). We don't want to combine two checks in one.<br>
</blockquote><div> </div><div>Agreed - Legal was a poor choice of terms. I don't know if we want to check TLI.isOperationLegal(...) though - an llvm.fma.* call (which is what we lower to) will always be legal, the only thing we care about is whether it's cheap right? (Which boils down to whether the target has an efficient fma instruction).</div>
<div><br></div><div>- Lang.</div></div>