
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

</head>

<body dir="ltr">

<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr">

<p>Thank you both for the test case. I'm working on the fix.</p>

<p><br>

</p>

<p>-Aditya</p>

<br>

<br>

<div style="color: rgb(0, 0, 0);">

<div>

<hr tabindex="-1" style="display:inline-block; width:98%">

<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> davide.italiano@gmail.com <davide.italiano@gmail.com> on behalf of Davide Italiano <davide@freebsd.org><br>

<b>Sent:</b> Wednesday, April 26, 2017 9:28 PM<br>

<b>To:</b> Chandler Carruth<br>

<b>Cc:</b> Aditya K; llvm-commits; Geoff Berry; Sebastian Pop<br>

<b>Subject:</b> Re: [llvm] r300200 - Re-apply "[GVNHoist] Move GVNHoist to function simplification part of pipeline."</font>

<div> </div>

</div>

</div>

<font size="2"><span style="font-size:10pt;">

<div class="PlainText">On Wed, Apr 26, 2017 at 5:01 PM, Chandler Carruth via llvm-commits<br>

<llvm-commits@lists.llvm.org> wrote:<br>

> Finally got a good test case for this:<br>

><br>

> <a href="http://llvm.org/PR32821" id="LPlnk593986" previewremoved="true">http://llvm.org/PR32821</a><br>

><br>

> I'm going to flip the flag to "off" as suggested by Geoff, I completely<br>

> agree with that rather than moving it around.<br>

><br>

<br>

Another testcase where GVNHoist makes bad choices<br>

<a href="https://bugs.llvm.org/show_bug.cgi?id=32811" id="LPlnk594793" previewremoved="true">https://bugs.llvm.org/show_bug.cgi?id=32811</a><br>

<br>

Thanks,<br>

<br>

-- <br>

Davide<br>

<br>

"There are no solved problems; there are only problems that are more<br>

or less solved" -- Henri Poincare<br>

</div>

</span></font></div>

</div>

</body>

</html>