aaron.ballman added a comment. In D102261#2754289 <https://reviews.llvm.org/D102261#2754289>, @tschuett wrote:
> My bad. My last comment: > > LangOptions::SYCL::Ver2017 > LangOptions::SYCL_2017 To me, those convey the same amount of information, so the use of the scoped enum doesn't get us much (but it would mean we can't use `LangOptions::SYCL` for any other purpose in the future). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102261/new/ https://reviews.llvm.org/D102261 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
