erichkeane added a comment.
LGTM, but want others to look first.
================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:481
Builder.defineMacro("CL_SYCL_LANGUAGE_VERSION", "121");
+ Builder.defineMacro("SYCL_LANGUAGE_VERSION", "201707");
+ } else if (LangOpts.getSYCLVersion() == LangOptions::SYCL_2020) {
----------------
This seems very related, but perhaps a part of a different patch? I think I'm
probably OK bringing this in during this patch, but it isn't strictly related
to adding 2020 support.
Repository:
rG LLVM Github Monorepo
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
