[PATCH] D52339: Support enums with a fixed underlying type in all language modes
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 28 11:14:11 PDT 2018
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D52339#1249432, @erik.pilkington wrote:
> Ping! This doesn't really affect CUDA or OpenCL more than any change to the compiler, so I don't think it makes sense to block this until we hear from them.
It impacts them insomuch as they're standards and they may want to opt out of extensions for keeping code portable to other compilers.
However, short of that concern, this LGTM. I've spoken with the paper author for the WG14 paper and I think this approach fits their general proposal, so I don't think we'll have divergence here.
Repository:
rC Clang
https://reviews.llvm.org/D52339
More information about the cfe-commits
mailing list