[PATCH] D33904: Add a __has_attribute_enhancement macro to clang

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 14 16:38:02 PDT 2017


aaron.ballman added a comment.

In https://reviews.llvm.org/D33904#778165, @george.burgess.iv wrote:

> > Why not just use __has_feature(overloadable_unmarked) or similar?
>
> My impression was that `__has_feature` was was for larger features than tweaks to attributes. If this would be an appropriate use of `__has_feature`, though, I'm happy to keep things simple.
>
> I'll update the other review with `__has_feature`. If it goes in with that, I'll abandon this.


I agree, this need seems more in line with `__has_feature` than adding a new feature testing macro.

> Thanks!




https://reviews.llvm.org/D33904





More information about the cfe-commits mailing list