[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