[llvm] [llvm-objdump] Remove some unneeded headers. (PR #120541)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 19 03:14:43 PST 2024
jh7370 wrote:
> Almost all headers I removed are included by the relative header. For example, StringSet is included by llvm-objdump.h. It’s the first time I submit my patch to community, sorry if I did something wrong🥺.
Don't worry, you've done nothing wrong! I just needed to understand the context to be able to focus my review appropriately. I'll try to look at this in the next couple of workdays, but I've got a couple of higher priority items to look at before Christmas, so may not get to it. Somebody else is welcome to review it instead, if they have time!
https://github.com/llvm/llvm-project/pull/120541
More information about the llvm-commits
mailing list