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

[SYCL][L0] Support for >4Gb device allocations. #298

Merged
merged 3 commits into from
Jun 3, 2021
Merged

[SYCL][L0] Support for >4Gb device allocations. #298

merged 3 commits into from
Jun 3, 2021

Conversation

smaslov-intel
Copy link

@smaslov-intel smaslov-intel commented May 27, 2021 •
edited
Loading

E2E test for intel/llvm#3840
Signed-off-by: Sergey V Maslov [email protected]

@smaslov-intel smaslov-intel requested a review from againull May 27, 2021 22:57
@smaslov-intel smaslov-intel mentioned this pull request May 27, 2021
Merged
@smaslov-intel smaslov-intel requested a review from jbrodman as a code owner June 1, 2021 17:40
@vladimirlaz
Copy link

@smaslov-intel could you please fix CI issues?

@smaslov-intel
Copy link
Author

@smaslov-intel could you please fix CI issues?

This is expected result with the existing compiler:

[2021-06-02T05:34:00.998Z] ********************
[2021-06-02T05:34:00.998Z] Failed Tests (1):
[2021-06-02T05:34:00.998Z] SYCL :: Plugin/max_malloc.cpp
[2021-06-02T05:34:00.998Z]
[2021-06-02T05:34:00.998Z] ********************
[2021-06-02T05:34:00.998Z] Unexpectedly Passed Tests (1):
[2021-06-02T05:34:00.998Z] SYCL :: USM/badmalloc.cpp

After intel/llvm#3840 was merged (yesterday) the testing should be clean.
@vladimirlaz : What is the proper way to re-trigger testing of this PR with the new compiler?

@vladimirlaz vladimirlaz merged commit 46fe69c into intel:intel Jun 3, 2021
smaslov-intel added a commit to smaslov-intel/llvm-test-suite that referenced this pull request Aug 12, 2021
@smaslov-intel
[SYCL][L0] Support for >4Gb device allocations. (intel#298)
69213e9 
Signed-off-by: Sergey V Maslov <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
@smaslov-intel
[SYCL][L0] Support for >4Gb device allocations. (intel/llvm-test-suit…
e020912 
…e#298)

Signed-off-by: Sergey V Maslov <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@againull againull againull approved these changes

@jbrodman jbrodman Awaiting requested review from jbrodman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@smaslov-intel @vladimirlaz @againull