omtcyfz added inline comments. ================ Comment at: clang-rename/USRFinder.cpp:80 @@ -79,1 +79,3 @@ TypeBeginLoc, 0, Context.getSourceManager(), Context.getLangOpts()); + if (const auto *TemplateTypeParm = dyn_cast<TemplateTypeParmType>(Loc.getType())) { + return setResult(TemplateTypeParm->getDecl(), TypeBeginLoc, TypeEndLoc); ---------------- alexfh wrote: > It's not common to use braces around single-line `if` bodies in LLVM/Clang > code. I do that literally everywhere :D LLVM Code Style isn't against it and I thinks it is good in terms of readability.
https://reviews.llvm.org/D22853 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits