<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jan 6, 2011, at 2:45 PM, Duncan Sands wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Hi Eric,<br><br><blockquote type="cite">+std::pair<SDValue, SDValue> DAGTypeLegalizer::ExpandAtomic(SDNode *Node) {<br></blockquote><br>why not put this function in LegalizeIntegerTypes? It doesn't seem to be of<br>general utility. Are you planning to use it from some other LegalizeXYZTypes<br>file?<br><br><blockquote type="cite">+<span class="Apple-tab-span" style="white-space:pre"> </span>std::pair<SDValue, SDValue> ExpandAtomic(SDNode *Node);<br></blockquote><blockquote type="cite">+<br></blockquote><br>Then you wouldn't need this declaration either.<br></div></blockquote></div><br><div>That's fair, I'll do that :)</div><div><br></div><div>-eric</div></body></html>