<div dir="ltr">Sort of a bigger-picture question: does it make sense to warn about this? I thought that optnone was just for debugging; is there a use case where the user would put optnone on something when debugging and actually care about whether optnone is overriding something else?<div><br></div><div>-- Sean Silva</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 11, 2014 at 4:27 PM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Diagnose when attribute 'optnone' conflicts with attributes on a<br>
different declaration.<br>
<br>
I modeled this on how dllimport/dllexport are handled; specifically,<br>
I made 'optnone' continue to "win" over 'always_inline' and 'minsize'.<br>
This preserves the previous "winner" behavior but adds the diagnostics<br>
that were requested.<br>
<br>
I have two questions about all this.<br>
<br>
First, the diagnostics point to the attribute being ignored, but not<br>
to the attribute that caused it to be ignored.  Is that okay?  This<br>
would be a problem for dllimport/dllexport as well as the new cases.<br>
<br>
Second, now that there are 5 similar cases (dllimport v. dllexport,<br>
plus optnone v. always_inline/minsize) it seems like there could be<br>
an opportunity to refactor some of that diagnostic checking into a<br>
templated helper function, along the lines of mergeVisibilityAttr.<br>
Should I pursue that? (Not clear it's much of a win... but maybe.)<br>
<br>
Thanks,<br>
--paulr<br>
<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div></div>