[PATCH] D154950: [include-cleaner] Fix the `fixIncludes` API not respect main-file header.
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 11 05:05:14 PDT 2023
kadircet accepted this revision.
kadircet added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang-tools-extra/include-cleaner/unittests/AnalysisTest.cpp:303
- EXPECT_EQ(fixIncludes(Results, Code, format::getLLVMStyle()), R"cpp(
+ EXPECT_EQ(fixIncludes(Results, "d.cc", Code, format::getLLVMStyle()),
+R"cpp(#include "d.h"
----------------
can you also have a test in which we insert "d.h" and it gets put into the right place?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154950/new/
https://reviews.llvm.org/D154950
More information about the cfe-commits
mailing list