[polly] r248823 - Replace default destructors by {} destructors

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 15:12:19 PDT 2015


On 09/29/2015 10:12 PM, David Blaikie wrote:
> On Tue, Sep 29, 2015 at 12:54 PM, Tobias Grosser via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
>> Author: grosser
>> Date: Tue Sep 29 14:52:09 2015
>> New Revision: 248823
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=248823&view=rev
>> Log:
>> Replace default destructors by {} destructors
>>
>> Hope this fixes the buildbots for now.
>>
>
> Oh, sorry about that. Didn't see those buildbot failures.

No worries. This was a rather quick fix, so it is not minimal as you
noted below. However, I already was surprised that your original
change caused compilation failures, but onlyon older gcc versions.
In case you happen to understand the error message, would it make
sense to move back to a fixed version of your change?

Tobias


More information about the llvm-commits mailing list