<div dir="ltr">Not yet, I'm dealing with a couple other regressions at the moment, but I'm still planning to look at it asap.<div><br><div>Is a 32kb regression large?</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 8, 2019 at 6:35 AM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</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">FWIW we noticed a ~32 KB size regression on Chrome for Android when<br>
rolling past your revert. Did you make any progress on figuring out<br>
what's wrong with the lowering?<br>
<br>
On Tue, May 7, 2019 at 9:12 AM Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
><br>
> Sorry for the breakage.<br>
><br>
> Please let me know when you have any details about how this broke things.<br>
><br>
> On Mon, May 6, 2019 at 11:52 PM Jordan Rupprecht via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> ><br>
> > Author: rupprecht<br>
> > Date: Mon May  6 14:55:05 2019<br>
> > New Revision: 360086<br>
> ><br>
> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=360086&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=360086&view=rev</a><br>
> > Log:<br>
> > Revert "Re-commit r357452: SimplifyCFG SinkCommonCodeFromPredecessors: Also sink function calls without used results (PR41259)"<br>
> ><br>
> > This reverts r357452 (git commit 21eb771dcb5c11d7500fa6ad551c97a921997f05).<br>
> ><br>
> > This was causing strange optimization-related test failures on an internal test. Will followup with more details offline.<br>
> ><br>
> > Modified:<br>
> >     llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp<br>
> >     llvm/trunk/test/CodeGen/AArch64/max-jump-table.ll<br>
> >     llvm/trunk/test/CodeGen/AArch64/min-jump-table.ll<br>
> >     llvm/trunk/test/CodeGen/AArch64/win64-jumptable.ll<br>
> >     llvm/trunk/test/CodeGen/ARM/cmpxchg-idioms.ll<br>
> >     llvm/trunk/test/Transforms/SimplifyCFG/sink-common-code.ll<br>
</blockquote></div>