[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