[cfe-dev] Warning options table

[Sebastian Redl]

On Tue, 3 Mar 2009 17:53:38 -0800, Ted Kremenek <kremenek at apple.com> wrote:
> On Mar 3, 2009, at 5:39 PM, Chris Lattner wrote:
> 
>>
>> For that, it is probably best to just have:
>>
>> let Library = "sema" in {
>>
>> at the top of each per-library file, and } at the end.
>>
>> -Chris
> 
> Does having separate per-library files pose issues with implementing - 
> Wall?  That is, to implement -Wall, or any other option whose warnings  
> might cross  several libraries, don't we need the ability to  
> potentially know about all the warnings across Clang?  I think that  
> was the original motivation to put all the warnings in one TableGen  
> file.
> 

If we have the warning option definitions in a separate file anyway (and I
think we should, since warning options are orthogonal to which component
issues the warning), you can just have that file include all the diagnostic
definition files.

Sebastian