[llvm] [obj2yaml] Add Obj2Yaml ability to dump GOFF header records (PR #204194)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 3 01:45:31 PDT 2026
================
@@ -16,14 +16,18 @@
#include "llvm/ADT/DenseMap.h"
#include "llvm/ADT/IndexedMap.h"
+#include "llvm/ADT/SmallString.h"
----------------
jh7370 wrote:
I'm guessing all/most of these new includes aren't actually strictly needed in this header because other headers include them? If so, see https://llvm.org/docs/CodingStandards.html#include-as-little-as-possible, which says not to include them (I note this list isn't minimal as it is, but there's no point in making it worse).
https://github.com/llvm/llvm-project/pull/204194
More information about the llvm-commits
mailing list