[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