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

[John McCall]

On Aug 7, 2010, at 10:07 PM, Eli Friedman wrote:

> 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?

Yes, thanks!

John.