<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 30, 2015 at 1:47 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"><div class="HOEnZb"><div class="h5">On 09/30/2015 09:51 PM, David Blaikie wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Sep 29, 2015 at 11:04 PM, Tobias Grosser <<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 09/30/2015 12:56 AM, David Blaikie wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Sep 29, 2015 at 3:12 PM, Tobias Grosser <<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>><br>
wrote:<br>
<br>
On 09/29/2015 10:12 PM, David Blaikie wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Tue, Sep 29, 2015 at 12:54 PM, Tobias Grosser via llvm-commits <<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
<br>
Author: grosser<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Date: Tue Sep 29 14:52:09 2015<br>
New Revision: 248823<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=248823&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=248823&view=rev</a><br>
Log:<br>
Replace default destructors by {} destructors<br>
<br>
Hope this fixes the buildbots for now.<br>
<br>
<br>
Oh, sorry about that. Didn't see those buildbot failures.<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
No worries. This was a rather quick fix, so it is not minimal as you<br>
noted below. However, I already was surprised that your original<br>
change caused compilation failures, but onlyon older gcc versions.<br>
In case you happen to understand the error message, would it make<br>
sense to move back to a fixed version of your change?<br>
<br>
</blockquote>
<br>
<br>
Do you have a link (or just copy/paste inline) to the buildbot failure? I<br>
didn't spot it in any of the emails I got.<br>
<br>
</blockquote>
<br>
I added it in one of my previous commits that tried to unbreak the bot:<br>
<br>
     BlockGenerators.h:683:7: error: looser throw specifier for ‘virtual<br>
     polly::RegionGenerator::~RegionGenerator()’<br>
     BlockGenerators.h:164:11: error:   overriding ‘virtual<br>
polly::BlockGenerator::~BlockGenerator() noexcept<br>
     (true)`<br>
</blockquote>
<br>
<br>
Ah, did this fail after 248765 or only before? I removed the derived dtor<br>
(since it was unnecessary) in an attempt to fix this error. Sorry,<br>
should've called that out in my commit message and in a follow up to<br>
r248764.<br>
</blockquote>
<br></div></div>
This failed after 248764:<br>
<br>
<a href="http://lab.llvm.org:8011/builders/polly-amd64-linux/builds/34576" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/polly-amd64-linux/builds/34576</a><br>
<br>
and remained broken with 248765:<br>
<br>
<a href="http://lab.llvm.org:8011/builders/polly-amd64-linux/builds/34577" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/polly-amd64-linux/builds/34577</a></blockquote><div><br></div><div>Hmm - beats me then... <br><br>- Dave</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Best,<br>
Tobias<br>
<br>
</blockquote></div><br></div></div>