[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.
More information about the cfe-commits