[cfe-commits] r164104 - /cfe/trunk/include/clang/Sema/TemplateDeduction.h

[Richard Smith]

On Tue, Sep 18, 2012 at 3:45 PM, Matthieu Monrocq <
matthieu.monrocq at gmail.com> wrote:

>
>
> On Tue, Sep 18, 2012 at 2:30 AM, Craig Topper <craig.topper at gmail.com>wrote:
>
>> Author: ctopper
>> Date: Tue Sep 18 00:30:57 2012
>> New Revision: 164104
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=164104&view=rev
>> Log:
>> Remove an unused private field exposed by the recent
>> LLVM_DELETED_FUNCTION changes.
>>
>> Modified:
>>     cfe/trunk/include/clang/Sema/TemplateDeduction.h
>>
>> Modified: cfe/trunk/include/clang/Sema/TemplateDeduction.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Sema/TemplateDeduction.h?rev=164104&r1=164103&r2=164104&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/include/clang/Sema/TemplateDeduction.h (original)
>> +++ cfe/trunk/include/clang/Sema/TemplateDeduction.h Tue Sep 18 00:30:57
>> 2012
>> @@ -28,9 +28,6 @@
>>  /// deduction, whose success or failure was described by a
>>  /// TemplateDeductionResult value.
>>  class TemplateDeductionInfo {
>> -  /// \brief The context in which the template arguments are stored.
>> -  ASTContext &Context;
>> -
>>    /// \brief The deduced template argument list.
>>    ///
>>    TemplateArgumentList *Deduced;
>> @@ -51,7 +48,7 @@
>>
>>  public:
>>    TemplateDeductionInfo(ASTContext &Context, SourceLocation Loc)
>> -    : Context(Context), Deduced(0), Loc(Loc), HasSFINAEDiagnostic(false)
>> { }
>> +    : Deduced(0), Loc(Loc), HasSFINAEDiagnostic(false) { }
>>
>> Looks to me like Context now is an unused argument.
>

Also the FIXMEs referring to the removed variable should be removed. They
don't seem to be marking something worth fixing any more.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120918/14a2acce/attachment.html>