Skip to content

Conversation

@aelovikov-intel
Copy link
Contributor

SYCL 1.2 support has been removed quite some time ago, FE only reports 202012L

getSYCLVersionMacros(const LangOptions &LangOpts) {
if (LangOpts.getSYCLVersion() == LangOptions::SYCL_2020)
return {{"SYCL_LANGUAGE_VERSION", "202012L"}};
llvm_unreachable("SYCL standard should be set");
}

so we can expect it's at least that.

SYCL 1.2 support has been removed quite some time ago, FE only reports
`202012L`

https://github.com/intel/llvm/blob/9e38e3a05f42b32819a4a570973b925df97e174b/clang/lib/Basic/Version.cpp#L129-L133

so we can expect it's at least that.
@aelovikov-intel
Copy link
Contributor Author

Looks like we relied on it not being set in two scenarios:

  • unittest (easy to fix by setting __SYCL_DISABLE_PARALLEL_FOR_RANGE_ROUNDING__ and SYCL2020_DISABLE_DEPRECATION_WARNINGS)
  • custom host compiler (that doesn't seem to have SYCL_LANGUAGE_VERSION)

I'm not sure what the solution for the second one should be, so I'm going to drop this PR for now.

@aelovikov-intel aelovikov-intel deleted the sycl-lang-ver branch April 3, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants