[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