Skip to content

Conversation

@WeiqunZhang
Copy link
Member

@WeiqunZhang WeiqunZhang commented Jul 8, 2025

  • AOT can help expose issues earlier.
  • A header we use has been deprecated by Intel.
  • A big kernel has been split into three so that their kernel parameter size can be less than 2KB.

This can help expose issues earlier.
@WeiqunZhang WeiqunZhang changed the title [WIP] SYCL CI: Turn on AOT [WIP] SYCL: Turn on AOT in CI and Update MKL Jul 9, 2025

# try apt install up to five times, to avoid connection splits
# FIXME install latest version of IntelLLVM, Intel MKL
# after conflicts with openPMD are resolved
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ax3l I updated the code that uses MKL FFT in this PR. What were the conflicts with openPMD?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay it was added in #5419. Since there is no error now, it is safe to remove it.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can also try to remove -O1 -NDEBUG, which was added in #3478 to an address oneAPI 2022.2.0 issue.

@WeiqunZhang WeiqunZhang changed the title [WIP] SYCL: Turn on AOT in CI and Update MKL SYCL: Turn on AOT in CI and Update MKL Jul 9, 2025
@WeiqunZhang
Copy link
Member Author

Do we know why this test test_rz_langmuir_multi_psatd_JRhom_LL4 failed? This PR could not be the reason.

Copy link
Member

@roelof-groenewald roelof-groenewald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding the AOT flag and fixing the large kernel!

@roelof-groenewald roelof-groenewald added component: tests Tests and CI backend: sycl Specific to DPC++/SYCL execution (CPUs/GPUs) labels Jul 9, 2025
@WeiqunZhang WeiqunZhang merged commit ffd299c into BLAST-WarpX:development Jul 9, 2025
50 checks passed
@WeiqunZhang WeiqunZhang deleted the sycl_ci_aot branch July 9, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend: sycl Specific to DPC++/SYCL execution (CPUs/GPUs) component: tests Tests and CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants