[llvm-commits] missing #includes in header files

[don hinton]

On Aug 21, 2012, at 15:29, David Blaikie <dblaikie at gmail.com> wrote:

> On Tue, Aug 21, 2012 at 8:53 AM, don hinton <don.hinton at gmx.com> wrote:
>> Hi:
>> 
>> While running clang_doc on the llvm/clang source tree, including individual header files, I've found a number of headers that don't include all the files they need.
> 
> What constraint are you testing that's being violated? You're trying
> to parse headers standalone & finding they don't have their
> dependencies (even though all actual source files that include these
> headers happen to have the extra dependencies included before them)?

Yes, exactly. (at least for the ones that make sense, not all do).

> 
> Makes sense - and yes, I'm sure any patches that make headers
> appropriately self-standing would be appreciated.

It's only a few, so I'll try to put it together tomorrow.  The bigger task would be to remove unnecessary ones, but it doesn't support that yet.

thanks...
don

> 
>> I'm making the changes locally in order to test my tool, and can post a patch later if you're interested.
>> 
>> thanks...
>> don
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits