hokein added inline comments.
================ Comment at: clang-tools-extra/clangd/unittests/RenameTests.cpp:371 auto TU = TestTU::withCode(Code.code()); + TU.AdditionalFiles["foo.inc"] = R"cpp( + #define Macro(X) X ---------------- ilya-biryukov wrote: > Maybe test this separately? It's only used in one test. > Having this added for all tests cases is somewhat confusing. moved to a separate test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69934/new/ https://reviews.llvm.org/D69934 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits