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

[SYCL] Removed handler::codeplay_host_task #428

Merged
merged 3 commits into from
Aug 31, 2021

Conversation

maximdimakov
Copy link

@maximdimakov maximdimakov commented Aug 30, 2021

handler::codeplay_host_task is deprecated and is removed in the patch intel/llvm#4426
Signed-off-by: mdimakov [email protected]

@@ -98,11 +94,11 @@ template <bool UseSYCL2020HostTask> void Thread1Fn(Context *Ctx) {
GeneratorAcc[Idx] = Idx;
};

CGH.single_task<NameGen<class Gen, UseSYCL2020HostTask>>(GeneratorKernel);
CGH.single_task<NameGen<class Gen, true>>(GeneratorKernel);

Choose a reason for hiding this comment

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

Does second template parameter still required in NameGen? Can it be removed?

Copy link
Author

Choose a reason for hiding this comment

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

I removed it

@dm-vodopyanov
Copy link

handler::codeplay_host_task is deprecated and will be removed

Does it mean that this patch removes handler::codeplay_host_task in this patch? If yes, it should be smth like: "and will be removed" -> "and is removed in this patch"

@maximdimakov
Copy link
Author

@dm-vodopyanov I changed the comment

Copy link

@dm-vodopyanov dm-vodopyanov left a comment

Choose a reason for hiding this comment

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

LGTM

@vladimirlaz vladimirlaz merged commit 446adc0 into intel:intel Aug 31, 2021
romanovvlad pushed a commit to intel/llvm that referenced this pull request Sep 9, 2021
handler::codeplay_host_task is deprecated in SYCL 2020.
This patch depends on :
Changes in Lit Tests : intel/llvm-test-suite#428 and intel/llvm-test-suite#451
Signed-off-by: mdimakov [email protected]
DoyleLi pushed a commit to DoyleLi/intel_llvm that referenced this pull request Sep 13, 2021
handler::codeplay_host_task is deprecated in SYCL 2020.
This patch depends on :
Changes in Lit Tests : intel/llvm-test-suite#428 and intel/llvm-test-suite#451
Signed-off-by: mdimakov [email protected]
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.

3 participants