-
Notifications
You must be signed in to change notification settings - Fork 771
[cpp.cond] Missing __has_cpp_attribute(assume)
CWG2615
#5740
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@jensmaurer Has the feature test macro been discussed? |
The change history of the paper says
I don't remember any discussion about a feature-test macro in CWG, and the CWG minutes don't show any discussion either. I presume everybody assumed |
Thanks -- I don't have a problem adding it, but there has been pushback on other missing macros being "normative changes that should go through the regular issue process". Maybe we can make this a CWG issue then? |
(Please relabel/reassign as you see fit!) |
__has_cpp_attribute(assume)
__has_cpp_attribute(assume)
CWG2615
P1774R8 adds the assumption attribute
assume
, but does not modify Table 22 "__has_cpp_attribute
values" to handle__has_cpp_attribute(assume)
.This appears to be an obvious mistake. Can it be fixed editorially?
The text was updated successfully, but these errors were encountered: