ABataev added inline comments.
================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:2549
+ Opts.SYCLIsDevice = Opts.SYCL && Args.hasArg(options::OPT_fsycl_is_device);
+ if (Opts.SYCL || Opts.SYCLIsDevice) {
+ // -sycl-std applies to any SYCL source, not only those containing kernels,
----------------
Just `Opt.SYCL` should be enough
================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:455
+ // SYCL Version is set to a value when building SYCL applications
+ switch (LangOpts.getSYCLVersion()) {
+ case LangOptions::SYCLVersionList::SYCL_2015:
----------------
Must be guarded with `LangOpts.SYCL`:
```
if (LangOpts.SYCL) ...
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72857/new/
https://reviews.llvm.org/D72857
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
