<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 30, 2016 at 12:27 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi there,<br>
<div><div class="h5">On 30 May 2016 at 18:28, ⚛ <<a href="mailto:0xe2.0x9a.0x9b@gmail.com">0xe2.0x9a.0x9b@gmail.com</a>> wrote:<br>
> This patch enables compilation with -flto.<br>
><br>
> The performance benefits of LTO (GCC 4.9 & 6.1) are about 1% for glxgears.<br>
> Performance changes in OpenGL games haven't been measured yet, my feeling is<br>
> that they are negligible.<br>
><br>
</div></div>Please try to send patches via git send-email for the future.<br>
<br>
While not a hard requirement, it would be nice if you setup your email<br>
client/git to show your name/nickname.<br>
It's always better to address someone directly, as opposed to "hey you".<br>
<br>
About the patch itself:<br>
 - Please add a summary message for the change. Peter's has a nice<br>
article on the topic [1].<br>
 - We normally try to include performance metrics in the commit<br>
message. It makes it easier to justify said change.<br>
 - Have you tried debugging the resulting binary ? As Matt mentioned<br>
there used to be issues with LTO, so if those are still unresolved we<br>
want to _avoid_ forcing that for everyone.<br></blockquote><div><br></div><div>Regarding the above two comments:  The patch he attached does not enable LTO, it just disables it for mapi so that *if* you put -flto in your CFLAGS, it doesn't blow up.  If this patch were enabling LTO by default, questions about performance and debugability would be reasonable.  However, given that the patch is purely protective, the only real question required to justify the patch is "is LTO actually a problem for mapi?"<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 - Are you use the mapi hunk is needed ? last time I've tried (some<br>
months ago, copying the tweaks from the Clearlinux spec file [2])<br>
things built without issues.<br></blockquote><div><br>You are claiming it isn't.<br><br></div><div>That said, I tried to build with -flto recently and it blew up.  I don't remember exactly why.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
Emil<br>
<br>
[1] <a href="http://who-t.blogspot.co.uk/2009/12/on-commit-messages.html" rel="noreferrer" target="_blank">http://who-t.blogspot.co.uk/2009/12/on-commit-messages.html</a><br>
[2] <a href="https://download.clearlinux.org/releases/8470/clear/source/SRPMS/" rel="noreferrer" target="_blank">https://download.clearlinux.org/releases/8470/clear/source/SRPMS/</a><br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>