-
Notifications
You must be signed in to change notification settings - Fork 246
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
[Core] Adding support for Intel LLVM compiler (aka ICX, aka DPC++) #12880
base: master
Are you sure you want to change the base?
Conversation
This reverts commit d89f983.
…-dpcpp-windows' of https://github.com/KratosMultiphysics/Kratos into core/support-for-dpcpp-windows
This reverts commit 8d0fd38.
…g _VA_ARGS_ as a token (##)
…ratosMultiphysics/Kratos into Core/support-for-dpcpp-windows
We may think to add the compìler to the CI? |
Maybe replacing the just core compilation? |
This reverts commit c199377.
Just as track, currently we have an hard blocker in the linking stage as we are exceeding the number of exposed symbols allowed by the linker. Should be possible to overcome with |
Looks like the abbys is looking us... |
📝 Description
This PR involves changes to the
kratos_export_api.h
file:Header has been cleaned up.
The header guard (
#ifndef KRATOS_EXPORT_API_H
) was replaced with a#pragma once
directive, which is a modern and more efficient way to prevent multiple inclusions of the header.Conditional declarations of explicit template instances (
KRATOS_API_EXTERN
) were updatedto accommodate Intel LLVM (aka ICX, aka DPC++) compilers on Windows.For now compilation only works with Ninja, settings must be as following:
Thanks @roigcarlo
🆕 Changelog