sammccall added inline comments.
================
Comment at: clang-tools-extra/include-cleaner/unittests/AnalysisTest.cpp:33
TEST(WalkUsed, Basic) {
// FIXME: Have a fixture for setting up tests.
llvm::Annotations Code(R"cpp(
----------------
sammccall wrote:
> @kadircet gentle reminder - you wanted to defer this, this patch adds a
> copy/paste of the test setup - want me to try to pull something out before
> this gets out of hand?
disregard, I misread this, the new test isn't calling walkUsed at all so
belongs somewhere else
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137320/new/
https://reviews.llvm.org/D137320
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
