[PATCH] D51292: [docs] Update clang-rename documentation

Kirill Bobyrev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 27 08:48:50 PDT 2018


kbobyrev added inline comments.


================
Comment at: clang-tools-extra/docs/clang-rename.rst:28
+:program:`clang-rename` infrastructure to handle renaming requests. Because of
+much better editor integration and support, it is advised to use
+:program:`clangd-rename` as part of :program:`clangd`. However, it is possible
----------------
ilya-biryukov wrote:
> I would not recommend `clangd` at this point, it only supports single-file renames, while `clang-rename` can actually do cross-TU renames IIUC.
> Not opposed to putting recommendations of using `clangd`, of course, but let's be explicit about the limitations.
Interesting, I thought we do the same logic in Clangd. Yeah, I think we should fix that behavior.


https://reviews.llvm.org/D51292





More information about the cfe-commits mailing list