-
Notifications
You must be signed in to change notification settings - Fork 115
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
Remove libc++
from tested Standard C++ APIs
#2080
base: main
Are you sure you want to change the base?
Conversation
libc++
from tested APIlibc++
from tested Standard C++ APIs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fair to me. I confirm that libc++ is not regularly tested.
We can update the table again after testing it manually or setting up automatic testing.
I would suggest waiting for others' opinions, though.
Was this gap introduced when the clang++ driver was removed from the DPC++ compiler distribution? I think in order to test with the clang++ driver we would need to use a build of the https://github.com/intel/llvm. |
https://www.intel.com/content/www/us/en/developer/articles/troubleshooting/sourcing-oneapi-overrides-existing-clang-drivers.html : "The oneAPI provided clang/clang++ drivers were not intended to be used by customers..." |
@timmiesmith can this patch be retargeted to 2022.8 (and the respective branch) instead, or is it already late? |
I think we're too late for inclusion in the oneAPI 2025.1 release, but we can pull this into the 2022.8.0 release branch so the open source release has the correct status. |
I have prepared separate PR #2086 for 2022.8 |
9c1314f
to
442e5e0
Compare
…api.rst - remove libc++ standard library from tested API
442e5e0
to
20c144d
Compare
Remove
libc++
tested Standard C++ APIs as not tested.Probably we will restore it when we add
xpu_api
tests for this standard library together withicpx
compiler.