Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Add oneAPI compilers (Intel/LLVM) to testing matrix. #542

Merged
merged 20 commits into from
Dec 15, 2021

Conversation

kris-rowe
Copy link
Member

Description

  • Adds the oneAPI compilers to the testing matrix used by GitHub.
  • The context added runs on the latest ubuntu using the CMake build.
  • Compilers are retrieved using apt.
  • Enables CPU testing of the OpenMP, OpenCL, and DPC++ modes
  • Adds the OpenCL and DPC++ modes to codecov

@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #542 (72065ab) into main (2abc3e9) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #542      +/-   ##
==========================================
- Coverage   76.36%   76.35%   -0.01%     
==========================================
  Files         262      262              
  Lines       19523    19521       -2     
==========================================
- Hits        14908    14906       -2     
  Misses       4615     4615              
Impacted Files Coverage Δ
src/occa/internal/bin/occa.cpp 37.20% <100.00%> (ø)
src/occa/internal/lang/modes/dpcpp.cpp 80.37% <100.00%> (ø)
src/occa/internal/utils/sys.cpp 76.51% <0.00%> (-0.11%) ⬇️
src/occa/internal/lang/specialMacros.cpp 60.92% <0.00%> (ø)

@kris-rowe kris-rowe requested a review from dmed256 November 23, 2021 18:41
@SFrijters
Copy link
Contributor

Just an additional note: I've recently started preparing our in-house libraries for IntelLLVM as well and one of the issues I ran into is that CMake doesn't properly detect the compiler before 3.20 (https://cmake.org/cmake/help/latest/release/3.20.html#compilers). I've added a compatibility workaround to our CMake, maybe that could be useful for occa as well if occa needs compiler-specific flags at some point.

@kris-rowe kris-rowe force-pushed the ci-add-oneapi-compilers branch from 060232f to 34223e3 Compare December 7, 2021 18:28
@kris-rowe kris-rowe merged commit 7dd3c62 into libocca:main Dec 15, 2021
@kris-rowe kris-rowe deleted the ci-add-oneapi-compilers branch December 15, 2021 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants