<div dir="ltr"><div>Thanks for letting me know. Reverted:</div><div><a href="https://reviews.llvm.org/rG07d549bce94f" rel="noreferrer" target="_blank">https://reviews.llvm.org/rG07d549bce94f</a></div><div><br></div><div>If you can get the IR for that file, that would be great. Otherwise, I'll try to set up a system to extract that.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 16, 2022 at 5:12 PM Florian Hahn <<a href="mailto:florian_hahn@apple.com" target="_blank">florian_hahn@apple.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"><div><div><div>Hi<br><div><br><blockquote type="cite"><div>On May 16, 2022, at 17:56, Sanjay Patel via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:</div><br><div><div><br>Author: Sanjay Patel<br>Date: 2022-05-16T12:55:52-04:00<br>New Revision: 3794cc0e996481e10307b67c8436aa44e0d65d22<br><br>URL: <a href="https://github.com/llvm/llvm-project/commit/3794cc0e996481e10307b67c8436aa44e0d65d22" target="_blank">https://github.com/llvm/llvm-project/commit/3794cc0e996481e10307b67c8436aa44e0d65d22</a><br>DIFF: <a href="https://github.com/llvm/llvm-project/commit/3794cc0e996481e10307b67c8436aa44e0d65d22.diff" target="_blank">https://github.com/llvm/llvm-project/commit/3794cc0e996481e10307b67c8436aa44e0d65d22.diff</a><br><br>LOG: [InstCombine] invert canonicalization for cast of signbit test<br><br>The existing transform was wrong in 3 ways:<br>1. It created an extra instruction when the source and dest types don't match.<br>2. It did not account for an extra use of the icmp, so could create 2 extra insts.<br>3. It favored bit hacks over icmp (icmp generally has better analysis).<br><br>This fixes #54692 (modeled by the PhaseOrdering tests).<br><br>This is a minimal step to fix the bug, but we should likely invert<br>the sibling transform for the "is negative" pattern too.<br><br>The backend should be able to invert this back to a shift if that<br>leads to better codegen.<br></div></div></blockquote></div><br><div><br></div><div>I believe this change may be causing a hang in bootstrap builds when building llvm-project/clang/unittests/Frontend/CompilerInvocationTest.cpp during stage2. With this patch reverted, building that file completes within 5 seconds. With the patch, Clang spins in instcombine.</div><div><br></div><div>It looks like the following bots are timing out because of this:</div><div><br></div><div><a href="https://lab.llvm.org/buildbot#builders/179/builds/3688" target="_blank"><font face="Arial, Helvetica, sans-serif" color="#1155cc">https://lab.llvm.org/buildbot#</font><font face="Arial, Helvetica, sans-serif" color="#1155cc">builders/179/builds/3688</font></a></div><div><a href="https://lab.llvm.org/buildbot#builders/85/builds/8641" target="_blank"><font face="Arial, Helvetica, sans-serif" color="#1155cc">https://lab.llvm.org/buildbot#</font><font face="Arial, Helvetica, sans-serif" color="#1155cc">builders/85/builds/8641</font></a></div><div><a href="https://lab.llvm.org/buildbot#builders/168/builds/6486" target="_blank"><font face="Arial, Helvetica, sans-serif" color="#1155cc">https://lab.llvm.org/buildbot#</font><font face="Arial, Helvetica, sans-serif" color="#1155cc">builders/168/builds/6486</font></a></div><div><a href="https://lab.llvm.org/buildbot#builders/74/builds/11107" target="_blank"><font face="Arial, Helvetica, sans-serif" color="#1155cc">https://lab.llvm.org/buildbot#</font><font face="Arial, Helvetica, sans-serif" color="#1155cc">builders/74/builds/11107</font></a></div><div><br></div><div>It might be good to revert the change if it cannot be fixed quickly, to get the bots back to green.</div><div><br></div><div>Cheers,</div><div>Florian</div></div></div></div></blockquote></div>