<div dir="ltr">Absolutely agreed. Thanks!<br><br>-eric<br></div><br><div class="gmail_quote">On Tue, Apr 28, 2015 at 8:41 AM Sanjay Patel <<a href="mailto:spatel@rotateright.com">spatel@rotateright.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D8900#162327" target="_blank">http://reviews.llvm.org/D8900#162327</a>, @echristo wrote:<br>
<br>
> It's still my preference if possible because it'll take the API nfc change out of the mix on the patch. Is it possible for you to do that?<br>
<br>
<br>
Hi Eric -<br>
<br>
Sure - it's a little more work, but it's the right thing to do. I think we should actually break this into 3 steps:<br>
<br>
1. Introduce the new Flags struct and have the existing BinaryWithFlagsSDNode use them only for nsw/nuw/exact (NFC).<br>
2. Convert BinaryWithFlagsSDNode to the broader SDNodeWithFlags class and extend the potential users to the FP nodes (NFC).<br>
3. Update DAGCombiner to use the 'arcp' flag and add the new test cases.<br>
<br>
First patch coming up...<br>
<br>
<br>
<a href="http://reviews.llvm.org/D8900" target="_blank">http://reviews.llvm.org/D8900</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div>