Diagnose when attribute 'optnone' conflicts with attributes on a
different declaration.

I modeled this on how dllimport/dllexport are handled; specifically,
I made 'optnone' continue to "win" over 'always_inline' and 'minsize'.
This preserves the previous "winner" behavior but adds the diagnostics
that were requested.

I have two questions about all this.

First, the diagnostics point to the attribute being ignored, but not
to the attribute that caused it to be ignored.  Is that okay?  This
would be a problem for dllimport/dllexport as well as the new cases.

Second, now that there are 5 similar cases (dllimport v. dllexport,
plus optnone v. always_inline/minsize) it seems like there could be
an opportunity to refactor some of that diagnostic checking into a
templated helper function, along the lines of mergeVisibilityAttr.
Should I pursue that? (Not clear it's much of a win... but maybe.)

Thanks,
--paulr

Attachment: optnone-part3.diff
Description: optnone-part3.diff

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to