I think r230448 tests this (windows was broken without this preparatory change).
On Tue, Feb 24, 2015 at 7:21 PM, David Blaikie <[email protected]> wrote: > test case? > > > On Tue, Feb 24, 2015 at 6:16 PM, Reid Kleckner <[email protected]> wrote: > >> Author: rnk >> Date: Tue Feb 24 20:16:02 2015 >> New Revision: 230445 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=230445&view=rev >> Log: >> Add missing MD canonicalization to MS vtable builder >> >> Modified: >> cfe/trunk/lib/AST/VTableBuilder.cpp >> >> Modified: cfe/trunk/lib/AST/VTableBuilder.cpp >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/VTableBuilder.cpp?rev=230445&r1=230444&r2=230445&view=diff >> >> ============================================================================== >> --- cfe/trunk/lib/AST/VTableBuilder.cpp (original) >> +++ cfe/trunk/lib/AST/VTableBuilder.cpp Tue Feb 24 20:16:02 2015 >> @@ -2930,6 +2930,7 @@ static void GroupNewVirtualOverloads( >> typedef llvm::DenseMap<DeclarationName, unsigned> >> VisitedGroupIndicesTy; >> VisitedGroupIndicesTy VisitedGroupIndices; >> for (const auto *MD : RD->methods()) { >> + MD = MD->getCanonicalDecl(); >> VisitedGroupIndicesTy::iterator J; >> bool Inserted; >> std::tie(J, Inserted) = VisitedGroupIndices.insert( >> >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
