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

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 28 01:09:59 PDT 2018


ilya-biryukov 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
----------------
kbobyrev wrote:
> 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.
For that we would need up-to-date cross references


https://reviews.llvm.org/D51292





More information about the cfe-commits mailing list