<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 20, 2015 at 2:24 PM, Tobias Grosser via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 10/20/2015 02:56 PM, Angel Garcia Gomez via cfe-commits wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: angelgarcia<br>
Date: Tue Oct 20 07:56:27 2015<br>
New Revision: 250824<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=250824&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=250824&view=rev</a><br>
Log:<br>
Apply modernize-use-default to clang-tools-extra.<br>
<br>
Summary: Replace empty bodies of default constructors and destructors with '= default'.<br>
</blockquote>
<br></span>
Hi Angel,<br>
<br>
this breaks <a href="http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3</a>. The builder runs gcc 4.7.2.</blockquote><div><br></div><div>Ah, I was/am seeing this on the GDB 7.5 buildbot too - but figured it might be GCC 4.6, because I couldn't reproduce similar behavior in small samples on GCC 4.7, but it might be that I have GCC 4.7.3 locally, not 4.7.2... <br><br>I think I have some ideas on how to fix this while keeping the change, but probably best to revert first, then try some things once we've got a reproduction locally to experiment with. (I think maybe Optional and possibly some other types are missing noexcept on their user-defined dtors, and GCC 4.old is assuming user defined dtors are noexcept(false), which is incorrect in C++11)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> This version is clearly not the most recent one, but still belongs to the set of gcc versions we currently document<br>
as supported:<br>
<br>
  <a href="http://llvm.org/docs/GettingStarted.html#software" rel="noreferrer" target="_blank">http://llvm.org/docs/GettingStarted.html#software</a><br>
<br>
Could you please revert this change.<br>
<br>
Thank you,<br>
Tobias<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>