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

[SYCL] Add testing for multi_ptr with extended USM address spaces #1439

Merged

Conversation

steffenlarsen
Copy link

@steffenlarsen steffenlarsen commented Dec 7, 2022

This commit splits multi_ptr and multi_ptr_legacy into two tests; one compiled and run as before and another defining
__ENABLE_USM_ADDR_SPACE__ to define the extended USM address spaces.

This requires intel/llvm#7680.

This commit splits multi_ptr and multi_ptr_legacy into two tests; one
compiled and run as before and another defining
__ENABLE_USM_ADDR_SPACE__ to define the extended USM address spaces.

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen steffenlarsen requested a review from a team as a code owner December 7, 2022 12:48
@steffenlarsen
Copy link
Author

/verify with intel/llvm#7680

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen
Copy link
Author

/verify with intel/llvm#7680

// RUN: %clangxx -D__ENABLE_USM_ADDR_SPACE__ -fsycl -fsycl-targets=%sycl_triple -fsycl-dead-args-optimization %s -o %t.out
// RUN: %ACC_RUN_PLACEHOLDER %t.out

//==--------------- multi_ptr.cpp - SYCL multi_ptr test --------------------==//

Choose a reason for hiding this comment

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

seems to be not updated.

Copy link
Author

Choose a reason for hiding this comment

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

Good eye! Updated.

@steffenlarsen steffenlarsen merged commit 4e3241c into intel:intel Jan 25, 2023
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
…tel#1439)

* [SYCL] Add testing for multi_ptr with extended USM address spaces

This commit splits multi_ptr and multi_ptr_legacy into two tests; one
compiled and run as before and another defining
__ENABLE_USM_ADDR_SPACE__ to define the extended USM address spaces.

Signed-off-by: Larsen, Steffen <[email protected]>

* Move to new namespace

Signed-off-by: Larsen, Steffen <[email protected]>

* Fix test license header

---------

Signed-off-by: Larsen, Steffen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…tel/llvm-test-suite#1439)

* [SYCL] Add testing for multi_ptr with extended USM address spaces

This commit splits multi_ptr and multi_ptr_legacy into two tests; one
compiled and run as before and another defining
__ENABLE_USM_ADDR_SPACE__ to define the extended USM address spaces.

Signed-off-by: Larsen, Steffen <[email protected]>

* Move to new namespace

Signed-off-by: Larsen, Steffen <[email protected]>

* Fix test license header

---------

Signed-off-by: Larsen, Steffen <[email protected]>
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