[cfe-commits] r148995 - in /cfe/trunk: lib/Sema/TreeTransform.h test/CodeGenCXX/c99-variable-length-array.cpp

Eli Friedman eli.friedman at gmail.com
Wed Jan 25 19:07:26 PST 2012


On Wed, Jan 25, 2012 at 6:13 PM, John McCall <rjmccall at apple.com> wrote:
> On Jan 25, 2012, at 3:20 PM, Eli Friedman wrote:
>> Author: efriedma
>> Date: Wed Jan 25 17:20:27 2012
>> New Revision: 148995
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=148995&view=rev
>> Log:
>> Don't stack-allocate an IntegerLiteral which can be referred to after the current method returns.  PR11744, part 2.
>
> It would be nice to avoid the completely useless allocation in the
> overwhelmingly dominant case where the element type isn't a VAT.

Yes... we should just use the expression from the type source info,
when that's available.

-Eli




More information about the cfe-commits mailing list