Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][CUDA] CXX std lib funcs for CUDA backend #1112

Merged
merged 3 commits into from
Sep 2, 2022

Conversation

hdelan
Copy link

@hdelan hdelan commented Jul 27, 2022

Relating to PR intel/llvm#6482

steffenlarsen pushed a commit to intel/llvm that referenced this pull request Sep 2, 2022
This PR allows CXX stdlib funcs to be used for NVPTX backend. 

See #6379

llvm-test-suite test: intel/llvm-test-suite#1112

It also adds the compiler flag `"-fbundle-no-offload-arch"`, which allows device code bundles to omit arch information, meaning a bundle with the tag `sycl-nvptx64-nvidia-cuda-sm_50` can be bundled with the tag `sycl-nvptx64-nvidia-cuda` when the flag is enabled. This allows for generic bc bundles for a given backend.

I have not moved the NVPTX complex implementation to libdevice. Let me know if I should do so.

The behaviour of `IsMathErrnoDefault()` now changes so that `-fmath-errno` is enabled by default for the CUDA toolchain. This allows libdevice funcs to be used by a similar exploit that the SYCL toolchain uses. If necessary this could be reset once the upstream discussion on NVPTX codegen for `llvm.sin` and other llvm builtins has been resolved and implemented. See https://discourse.llvm.org/t/nvptx-codegen-for-llvm-sin-and-friends/58170/11?u=jdoerfert
@steffenlarsen
Copy link

@hdelan - Could you please add a merge commit to rerun testing?

@hdelan
Copy link
Author

hdelan commented Sep 2, 2022

Yep done

Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

LGTM!

@steffenlarsen
Copy link

Windows failures are known and are unrelated.

@steffenlarsen steffenlarsen merged commit 3f3402c into intel:intel Sep 2, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants