Skip to content

Conversation

keyradical
Copy link
Contributor

A small fix for windows build which was failing due to some incorrect function pointer typedefs. Issue and fix are described here: intel/llvm#12710

intel/llvm CI: intel/llvm#12784

@keyradical
Copy link
Contributor Author

CI failure seems unrelated

@hdelan
Copy link
Contributor

hdelan commented Feb 22, 2024

Looks good. You can make this ready to merge

@keyradical keyradical added the ready to merge Added to PR's which are ready to merge label Feb 22, 2024
@kbenzie kbenzie added the v0.9.x Include in the v0.9.x release label Mar 11, 2024
@kbenzie
Copy link
Contributor

kbenzie commented Mar 11, 2024

Please pull in the main branch to have up to date testing, also update the tag in the intel/llvm PR.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 12.51%. Comparing base (78ef1ca) to head (0446c65).
Report is 120 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1368      +/-   ##
==========================================
- Coverage   14.82%   12.51%   -2.32%     
==========================================
  Files         250      239      -11     
  Lines       36220    35949     -271     
  Branches     4094     4076      -18     
==========================================
- Hits         5369     4498     -871     
- Misses      30800    31447     +647     
+ Partials       51        4      -47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kbenzie kbenzie merged commit 1cf9a08 into oneapi-src:main Mar 13, 2024
martygrant pushed a commit to intel/llvm that referenced this pull request Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Added to PR's which are ready to merge v0.9.x Include in the v0.9.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants