Skip to content

Conversation

hvdijk
Copy link
Contributor

@hvdijk hvdijk commented Jul 8, 2025

Make sure we create a NativeCPUABIInfo rather than a DefaultABIInfo. At the moment NativeCPUABIInfo behaves identically to DefaultABIInfo but upcoming changes are intended to deviate from the default ABI. This change is split out into a separate commit for easier review.

Make sure we create a NativeCPUABIInfo rather than a DefaultABIInfo. At
the moment NativeCPUABIInfo behaves identically to DefaultABIInfo but
upcoming changes are intended to deviate from the default ABI. This
change is split out into a separate commit for easier review.
@hvdijk hvdijk requested review from a team as code owners July 8, 2025 12:55
@hvdijk hvdijk requested a review from ldrumm July 8, 2025 12:55
@hvdijk hvdijk temporarily deployed to WindowsCILock July 8, 2025 13:01 — with GitHub Actions Inactive
@hvdijk hvdijk temporarily deployed to WindowsCILock July 8, 2025 13:34 — with GitHub Actions Inactive
@hvdijk hvdijk temporarily deployed to WindowsCILock July 8, 2025 13:34 — with GitHub Actions Inactive
Copy link
Contributor

@npmiller npmiller left a comment

Choose a reason for hiding this comment

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

libclc change LGTM

@hvdijk
Copy link
Contributor Author

hvdijk commented Jul 11, 2025

ping @intel/dpcpp-cfe-reviewers

@hvdijk
Copy link
Contributor Author

hvdijk commented Jul 14, 2025

@intel/llvm-gatekeepers This is ready to be merged, thanks.

@sarnex sarnex merged commit b902d64 into intel:sycl Jul 14, 2025
37 of 38 checks passed
@hvdijk hvdijk deleted the create-nativecpuabiinfo branch September 4, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants