[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