[cfe-commits] [PATCH] Use static storage duration for file-scope compound literals in C++ dynamic initializers

Richard Smith richard at metafoo.co.uk
Tue Nov 22 14:55:09 PST 2011


On Tue, November 22, 2011 22:39, Chandler Carruth wrote:
> On Tue, Nov 22, 2011 at 2:30 PM, Richard Smith <richard at metafoo.co.uk>wrote:
>> The attached patch fixes the compound-literals-in-C++ extension to always
>> use static storage duration for file-scope compound literals, as per GCC's
>> documentation for this extension. Currently we only give them static storage
>>  duration if the initializer in which they appear is a constant.
>>
>> OK to commit?
>
> It's not clear to me why we need the helper function? Otherwise this looks
> fine to me...

In r145087. We need the helper function because class ConstExprEmitter is
defined in that .cpp file.

Thanks!
Richard




More information about the cfe-commits mailing list