<div dir="ltr">Sorry, I have just seen all this. Thank you for reverting it, David.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 20, 2015 at 11:48 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Reverted in r250862</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 20, 2015 at 2:39 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 10/20/2015 11:37 PM, David Blaikie wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
<br>
<br>
On Tue, Oct 20, 2015 at 2:24 PM, Tobias Grosser via cfe-commits<br></span><span>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a> <mailto:<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>>> wrote:<br>
<br>
    On 10/20/2015 02:56 PM, Angel Garcia Gomez via cfe-commits wrote:<br>
<br>
        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<br>
        destructors with '= default'.<br>
<br>
<br>
    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>.<br>
    The builder runs gcc 4.7.2.<br>
<br>
<br>
Ah, I was/am seeing this on the GDB 7.5 buildbot too - but figured it<br>
might be GCC 4.6, because I couldn't reproduce similar behavior in small<br>
samples on GCC 4.7, but it might be that I have GCC 4.7.3 locally, not<br>
4.7.2...<br>
<br>
I think I have some ideas on how to fix this while keeping the change,<br>
but probably best to revert first, then try some things once we've got a<br>
reproduction locally to experiment with. (I think maybe Optional and<br>
possibly some other types are missing noexcept on their user-defined<br>
dtors, and GCC 4.old is assuming user defined dtors are noexcept(false),<br>
which is incorrect in C++11)<br>
</span></blockquote>
<br>
This seems to be the same issue (or at least a similar issue) to what we had seen in the Polly source. It would be interesting to understand what is going on.<br>
<br>
Best,<br>
Tobias<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>