[PATCH] D39120: [rename] Don't overwrite the template argument when renaming a template function.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 20 05:31:58 PDT 2017


ioeric added inline comments.


================
Comment at: lib/Tooling/Refactoring/Rename/USRLocFinder.cpp:227
+    SourceLocation EndLoc = Expr->hasExplicitTemplateArgs()
+                                ? Expr->getLAngleLoc().getLocWithOffset(-1)
+                                : Expr->getLocEnd();
----------------
I wonder what would happen if we have `foo       <int>()`?


https://reviews.llvm.org/D39120





More information about the cfe-commits mailing list