r230445 - Add missing MD canonicalization to MS vtable builder

[Nico Weber]

I think r230448 tests this (windows was broken without this preparatory
change).

On Tue, Feb 24, 2015 at 7:21 PM, David Blaikie <dblaikie at gmail.com> wrote:

> test case?
>
>
> On Tue, Feb 24, 2015 at 6:16 PM, Reid Kleckner <reid at kleckner.net> 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
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150224/b7435be3/attachment.html>