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

[Daniel Dunbar]

On Thu, Feb 11, 2010 at 9:55 AM, John McCall <rjmccall at apple.com> wrote:
> 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.

Sounds good!

 - Daniel

> John.
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>