<div dir="ltr">If we change the IRBuilder, that's not going to catch cases where a regression test already has the 'fsub -0.0', is it? I'd think that only comes into play if the regression test creates that op as part of its transformations.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 7, 2019 at 12:20 PM Cameron McInally <<a href="mailto:cameron.mcinally@nyu.edu">cameron.mcinally@nyu.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr">On Tue, May 7, 2019 at 11:44 AM Sanjay Patel <<a href="mailto:spatel@rotateright.com" target="_blank">spatel@rotateright.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 7, 2019 at 7:16 AM Kevin Neal <<a href="mailto:Kevin.Neal@sas.com" target="_blank">Kevin.Neal@sas.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
How should we handle regressions in code quality that are exposed by tests?<br></blockquote><div><br></div><div>One idea I had (or I may be remembering someone else's suggestion) was to grep the existing IR regression tests for 'fsub -0.0', then create a sibling test using fneg. If that doesn't result in the same output, then we need to do some pattern matching enhancement.<br></div></div></div></blockquote><div><br></div>I'm not opposed to this, but it does sound like a ton of work.<div><br></div><div>What if we checked for a -0.0 constant in IRBuilder::FSub*(...) and created an FNeg instead? Then we should only see test changes with `fsub -0.0`-> `fneg`. If we find anything else, it would need a closer look. Does that miss anything?</div></div></div></div>
</blockquote></div>