This is an archive of the discontinued LLVM Phabricator instance.

Add comments to the diagnostic kinds in Diagnostic.td.
ClosedPublic

Authored by thakis on Apr 21 2017, 2:00 PM.

Details

Reviewers
hans

Diff Detail

Event Timeline

thakis created this revision.Apr 21 2017, 2:00 PM

Not sure if this is useful, but this is where I always check first.

hans accepted this revision.Apr 21 2017, 2:05 PM

Highly useful I'd say!

lgtm

include/clang/Basic/Diagnostic.td
113

Nit: On first glance, one might parse this as the accepted language extensions being enabled by -pedantic, not the warnings. Maybe spell it out: "about accepted language extensions. The warnings are default-off bug enabled by -pedantic."

This revision is now accepted and ready to land.Apr 21 2017, 2:05 PM
thakis closed this revision.Apr 21 2017, 2:08 PM
thakis marked an inline comment as done.

r301039, thanks.

This comment was removed by chandlerc.
This comment was removed by chandlerc.
This comment was removed by chandlerc.