<div dir="ltr"><div style>Hi Tim,</div><div style><br></div><div style>A few questions...</div><div><br></div><div>+  return TLI.makeLibCall(DAG, GetFPLibCall(N->getValueType(0),</div><div>+                                           RTLIB::ADD_F32,</div>
<div>+                                           RTLIB::ADD_F64,</div><div>+                                           RTLIB::ADD_F80,</div><div>+                                           RTLIB::ADD_PPCF128),</div><div>+                         NVT, Ops, 2, false, N->getDebugLoc());</div>
<div><br></div><div style>Shouldn't this (and all other variations) also take into account the IEEE variation of long doubles?</div><div class="gmail_extra" style><br>As far as I can tell (I'm not a diff engine), the main move was softenSetCCOperands() and makeLibCall() going to TargetLowering. Your description doesn't tell why this is necessary at all and it's hard to point it out from the diff, since they're in different files and everything appears different, but look similar enough.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>If the move was just good, not necessary, it'd be easier to have it in two changes: one with the real change (with a small diff) and another with the code move (for the sake of simplicity and code organization).</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>cheers,</div><div class="gmail_extra" style>--renato</div></div>