[PATCH] D45444: [clang-tidy] implement new check for const-correctness

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 13 00:24:10 PDT 2018


JonasToth added a comment.

Thank you very much :)

Am 12.08.2018 um 00:17 schrieb Shuai Wang via Phabricator:

> shuaiwang added a comment.
> 
> In https://reviews.llvm.org/D45444#1196271, @JonasToth wrote:
> 
>> Always the same with the templates ;) So uninstantiated templates should
>> 
>>   just be ignored.
>> 
>> I think it would be better to have it in the ExprMutAnalyzer, because
>> 
>>   that part can not decide on const-ness. Fixing it here would just
>>   circumvent the problem but not fix it, would you agree?
> 
> Agreed :)
>  I'll create a diff handling various template related cases in ExprMutationAnalyzer.
> 
> Repository:
> 
>   rCTE Clang Tools Extra
> 
> https://reviews.llvm.org/D45444


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D45444





More information about the cfe-commits mailing list