Add 'remark' diagnostic type in clang

Chris Lattner clattner at apple.com
Wed Feb 26 15:20:38 PST 2014


On Feb 26, 2014, at 2:57 PM, Arthur O'Dwyer <arthur.j.odwyer at gmail.com> wrote:

> On Wed, Feb 26, 2014 at 2:27 PM, Chris Lattner <clattner at apple.com> wrote:
>> On Feb 26, 2014, at 2:26 PM, Hal Finkel <hfinkel at anl.gov> wrote:
>>> 
>>> Remarks should often indicate things that are good (like this loop was vectorized),
>>> and so upgrading remarks to errors will only make sense for 'negative' remarks.
>> 
>> Each remark should have its own identifier, so people can choose to upgrade them
>> or not.  It’s not worth the complexity to try to add another level or behavior for
>> “positive remarks”.
> 
> Hear hear hear hear hear.
> 
> -------------
> 
> Going off on an ambitious tangent here:

If we were able to remove the existing diagnostic flags, doing these changes might be nice.  However, since we can’t, this would just a lot of complexity without making the world better :-)

-Chris





More information about the cfe-commits mailing list