<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Oh, that won't work since BinaryOperator::CreateFSub*(...) returns a BinaryOperator. We'd need a UnaryOperator.</div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 7, 2019 at 3:41 PM Cameron McInally via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Ah, yeah. That was wrong. I should have said BinaryOperator::CreateFSub*(...).</div><div dir="ltr"><br></div><div></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 7, 2019 at 3:07 PM Sanjay Patel <<a href="mailto:spatel@rotateright.com" target="_blank">spatel@rotateright.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><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" target="_blank">cameron.mcinally@nyu.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color: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-width:1px;border-left-style:solid;border-left-color: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-width:1px;border-left-style:solid;border-left-color: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>
</blockquote></div></div>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=DwIGaQ&c=slrrB7dE8n7gBJbeO0g-IQ&r=O_4M49EtSpZ_-BQYeigzGv0P4__noMcSu2RYEjS1vKs&m=XVlEMuLhfcYrkLvZK3JkDVzubMrC9xr9ibDCwahmQYQ&s=Xl8DN1FWAqyKwnDHEfBLuO1r1s3bKRjm9tUGq3j1YBU&e=" rel="noreferrer" target="_blank">https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=DwIGaQ&c=slrrB7dE8n7gBJbeO0g-IQ&r=O_4M49EtSpZ_-BQYeigzGv0P4__noMcSu2RYEjS1vKs&m=XVlEMuLhfcYrkLvZK3JkDVzubMrC9xr9ibDCwahmQYQ&s=Xl8DN1FWAqyKwnDHEfBLuO1r1s3bKRjm9tUGq3j1YBU&e=</a><br>
</blockquote></div>