[llvm] [LLVM] [NFC] - Remove duplicate #include headers from the files of llvm dir (PR #141057)
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Thu May 22 10:05:14 PDT 2025
================
@@ -10,7 +10,6 @@
//
//===----------------------------------------------------------------------===//
-#include "llvm/IR/ProfDataUtils.h"
----------------
dwblaikie wrote:
The LLVM style guide ( https://llvm.org/docs/CodingStandards.html#include-style ) requires the main module header (the header that matches this cpp file) to go first - so could you remove the other instance instead of this one? (& add a gap/blank line between the main file header and the rest, this should prevent clang-tidy from complaining about header sort order, I think?)
Similarly anywhere else in this patch.
https://github.com/llvm/llvm-project/pull/141057
More information about the llvm-commits
mailing list