[PATCH] Target-specific attributes
Aaron Ballman
aaron at aaronballman.com
Thu Jan 9 15:04:28 PST 2014
Thanks! Committed in r198896.
On Thu, Jan 9, 2014 at 5:37 PM, Richard Smith <richard at metafoo.co.uk> wrote:
> LGTM, thanks!
>
>
> + default:
> + // Type attributes are handled elsewhere; silently move on.
> + assert(Attr.isTypeAttr() && "Non-type attribute not handled");
> + break;
>
> Eventually it'd be nice to be able to TableGen a list of attributes we don't
> expect to see here, so we can automatically generate case labels for them,
> and get a compile-time error if something is missing from this switch
> statement.
>
I agree, that's a good idea.
~Aaron
More information about the cfe-commits
mailing list