<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">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 class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
If you have any information on the reason for the regression(s), that would of course the great.<br></blockquote><div><br></div><div>I'll get this and look.</div><div><br></div><div>Thanks and sorry for the delay.</div><div><br></div><div>-eric </div></div></div>