[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