[PATCH] Automated handling of language options for attributes

Richard Smith richard at metafoo.co.uk
Mon Dec 2 11:27:11 PST 2013


I guess the intent is to eventually switch over to generating
include/clang/Basic/LangOptions.def with tablegen, with a shared .td file
being used for both that and the attributes?

Please name your generated functions with a leading lowercase letter
("defaultDiagnoseLangOpts", "checkCPlusPlusLangOpts"); other than that,
LGTM. Thank you!


On Mon, Dec 2, 2013 at 11:17 AM, Aaron Ballman <aaron at aaronballman.com>wrote:

> Several attributes test whether specific language options are enabled
> before performing the required semantic handling. This patch automates
> that task by pushing it into tablegen.
>
> I've added a list of language options that can be optionally specified
> for an attribute. If none of the given language options are enabled, a
> diagnostic is issued. This allowed several attributes to use the
> simple attribute handler.
>
> ~Aaron
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131202/e546399f/attachment.html>


More information about the cfe-commits mailing list