<div class="gmail_quote">On Tue, Jul 10, 2012 at 10:32 AM, Tanya Lattner <span dir="ltr"><<a href="mailto:lattner@apple.com" target="_blank">lattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">I've added a new warning for an OpenCL attribute and it needs to go under a flag. I noticed that in the Misc/warning-flags.c test there are a bunch of attribute warnings that do not have flags.<div>
<br></div><div>I think that the group "IgnoredAttribute" would apply to the warning I added and the following below as most are because the attribute is incorrectly applied and therefore ignored.</div><div><br></div>
<div>Please let me know what you think and I'll go clean these up. </div></div></blockquote><div><br></div><div>I think we'd benefit from a distinction between "your code is broken: the attribute is ignored because it is being used incorrectly" and "we think your code is fine but we do not support this attribute here" -- but we don't currently have such a distinction so I think the change you're proposing is fine.</div>
</div>