<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 25, 2021 at 1:27 AM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.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 dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 23, 2021 at 5:38 PM Nikita Popov via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.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">nikic added a comment.<br>
<br>
In D96881#2646035 <<a href="https://reviews.llvm.org/D96881#2646035" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96881#2646035</a>>, @echristo wrote:<br>
<br>
> In D96881#2627389 <<a href="https://reviews.llvm.org/D96881#2627389" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96881#2627389</a>>, @nikic wrote:<br>
><br>
>> LGTM per above discussion. It's not ideal, but I think it's the best we can do right now.<br>
><br>
> Hi All,<br>
><br>
> I'm seeing a rather numerous set of performance degredations and code size increases on x86 with this patch. I think that's a little unexpected, but do we have benchmarks and runs showing that this is a profitable change to happen in general?<br>
<br>
If you're seeing widespread regressions, let's revert this change. My assumption here was that it would fix the motivational case without impacting anything else much, but apparently that's not the case.<br>
<br></blockquote><div><br></div><div>We're working on that. Some of the size increases are small but numerous which makes smallish testcases hard. The impact may not be too large if it's worth the long term maintenance.</div></div></div></blockquote><div><br></div><div>Following back up here. I think we're ok here. Thanks for the offer and sorry it took me so long to follow back up.</div><div><br></div><div>-eric </div></div></div>