[cfe-commits] r95577 - in /cfe/trunk: lib/CodeGen/CodeGenModule.cpp test/CodeGenCXX/const-init.cpp

[John McCall]

On Feb 11, 2010, at 8:52 AM, Daniel Dunbar wrote:
> On Mon, Feb 8, 2010 at 1:46 PM, John McCall <rjmccall at apple.com> wrote:
>> Author: rjmccall
>> Date: Mon Feb  8 15:46:50 2010
>> New Revision: 95577
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=95577&view=rev
>> Log:
>> Emit global references with constant initializers as constants.  Fixes PR5585.
>> 
>> The standard actually says that such references should have internal linkage,
>> but gcc doesn't do that, so we probably can't get away with it.
> 
> Would it be worth trying?

Yep, that's a good idea.  I'd rather not try to diverge from gcc on things like linkage
while we're still bringing up C++, but I've filed PR6278 so we don't lose track of this.

John.