<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">There is at least a 3% regression on CINT2006/403.gcc. I guess this suggests there is enough evidence that the preferred default setting should be ‘true’ .<div class=""><div class=""><br class=""></div><div class="">Gerolf</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jul 21, 2015, at 2:48 PM, Sergey Ostanevich <<a href="mailto:sergos.gnu@gmail.com" class="">sergos.gnu@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Chandler,<div class=""><br class=""></div><div class="">We seen a number of regressions across EEMBC 1.1, 2.0 and SPEC 2000, 2006 tests using "<span style="font-family:Calibri,sans-serif;font-size:11pt;line-height:115%" class="">-Ofast
-flto -funroll-loops -mfpmath=sse" with targets "slm" and "core-avx2". </span></div><div class="">Will try to work out some small test case.</div><div class=""><br class=""></div><div class="">Sergos</div></div><div class="gmail_extra"><br class=""><div class="gmail_quote">On Fri, Jul 17, 2015 at 10:02 AM, Gerolf Hoflehner <span dir="ltr" class=""><<a href="mailto:ghoflehner@apple.com" target="_blank" class="">ghoflehner@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br class="">
> On Jul 16, 2015, at 11:48 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" class="">chandlerc@gmail.com</a>> wrote:<br class="">
><br class="">
> chandlerc marked 2 inline comments as done.<br class="">
> chandlerc added a comment.<br class="">
><br class="">
> Thanks Pete and Gerolf!<br class="">
><br class="">
> Based on Pete's review and some benchmark results Michael Z shared that show very limited impact of GlobalsModRef on performance, I'm submitting this as is. If folks *do* end up seeing regressions, by all means follow up here or on the commit thread and we can try other defaults, etc.<br class="">
><br class="">
><br class="">
> ================<br class="">
> Comment at: lib/Analysis/IPA/GlobalsModRef.cpp:56<br class="">
> @@ +55,3 @@<br class="">
> +static cl::opt<bool> EnableUnsafeGlobalsModRefAliasResults(<br class="">
> + "enable-unsafe-globalsmodref-alias-results", cl::init(false));<br class="">
> +<br class="">
> ----------------<br class="">
> Gerolf wrote:<br class="">
>> My preference for default is 'true'. Innocent until proven guilty - with test case counting as a proof. If performance data is in the noise I will switch my preference.<br class="">
> Michael Z got spec2k6 (and some other benchmarks) and some other numbers with GlobalsModRef completely disabled and the changes were in the noise, so I'm going with the default of false based on this comment.<br class="">
</span>Not sure. My understanding from earlier today was that were some issues with the runs that still need to be resolved.<br class="">
<span class="">><br class="">
> However, if you or anyone else gets performance data that is *not* in the noise, just shout, and I'm happy to flip this the other way. =]<br class="">
</span>Sounds great! But in case there is a loss the best outcome might be a small test case so we can brainstorm the best way to recover it.<br class="">
<div class="HOEnZb"><div class="h5">><br class="">
><br class="">
> <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11213&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=bLgorTc7FFgYrY_LD1jx7Lekt9aGEbieMqdK17uprAo&s=y0_ePF0Z6YyQjHJwu3NjQC0MMt6eN6hG2EV-PPBAlSw&e=" rel="noreferrer" target="_blank" class="">http://reviews.llvm.org/D11213</a><br class="">
><br class="">
><br class="">
><br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
llvm-commits mailing list<br class="">
<a href="mailto:llvm-commits@cs.uiuc.edu" class="">llvm-commits@cs.uiuc.edu</a><br class="">
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank" class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br class="">
</div></div></blockquote></div><br class=""></div>
</div></blockquote></div><br class=""></div></div></body></html>