[clang] [lld] [llvm] [mlir] [NFC][IWYU] Update Support library with IWYU. (PR #102707)

Jay Foad via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 12 07:50:00 PDT 2024


jayfoad wrote:

> > Did you know that LLVM intentionally does not follow IWYU and favors forward declarations: https://llvm.org/docs/CodingStandards.html#include-as-little-as-possible
> 
> Yes, but I actually do not see what part of the mentioned standard' section conflicts with the change.

Could you try preprocessing each affected source file (i.e. compile it with -E) and see whether the total number of included source lines goes up or down? I think that would help to demonstrate whether this patch violates the "include as little as possible" philosophy.

https://github.com/llvm/llvm-project/pull/102707


More information about the cfe-commits mailing list