[cfe-commits] r110526 - in /cfe/trunk: lib/Sema/SemaDeclCXX.cpp test/SemaCXX/virtual-base-used.cpp

Eli Friedman eli.friedman at gmail.com
Sat Aug 7 22:07:46 PDT 2010


On Sat, Aug 7, 2010 at 8:18 PM, John McCall <rjmccall at apple.com> wrote:
> On Aug 7, 2010, at 4:11 PM, Eli Friedman wrote:
>> Author: efriedma
>> Date: Sat Aug  7 18:11:44 2010
>> New Revision: 110526
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=110526&view=rev
>> Log:
>> PR7800: both virtual and non-virtual bases must be marked as used for VTTs.
>
> I think this is doing the right thing because of the recursion, but could you add
> a test case for indirect virtual bases?

Is r110540 sufficient?

-Eli




More information about the cfe-commits mailing list