[PATCH] D39120: [rename] Don't overwrite the template argument when renaming a template function.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 23 02:04:11 PDT 2017
hokein added inline comments.
================
Comment at: unittests/Rename/RenameFunctionTest.cpp:232
+ std::string Expected = R"(
+ namespace na {
+ template<typename T> T Y();
----------------
cierpuchaw wrote:
> Shouldn't this be `namespace nb {`?
It is intended. We don't change the namespace here, only the name of the definition will be changed.
Repository:
rL LLVM
https://reviews.llvm.org/D39120
More information about the cfe-commits
mailing list