[cfe-dev] Determine whether the current clang version has a specific bug
Daniel Jasper
djasper at google.com
Tue Jun 26 22:51:17 PDT 2012
Changed and submitted as r159252.
On Wed, Jun 27, 2012 at 3:03 AM, Jordan Rose <jordan_rose at apple.com> wrote:
> The __has_feature looks fine, but shouldn't you at least put an example in
> the test case?
>
> +class HasFeatureTest {
> +#if __has_feature(attribute_unused_on_fields)
> + int unused_; // expected-warning{{private field 'unused_' is not used}}
> + int unused2_ __attribute__((unused)); // no-warning
> +#endif
> +};
> +
>
> I know this is already covered by other tests, but it seems weird to test
> against a feature and then not use it.
>
> Jordan
>
> On Jun 26, 2012, at 3:06 PM, Daniel Jasper wrote:
>
> > Can someone please review this patch?
> >
> > Thank you!
> > Daniel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20120627/c4908dab/attachment.html>
More information about the cfe-dev
mailing list