[PATCH] D137320: [include-cleaner] Initial version for the "Location=>Header" step
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 11 03:35:42 PST 2022
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
More information about the cfe-commits
mailing list