Skip to content

[SYCL] Test for fix of linked alloca's deps #1470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bader merged 16 commits into from
Aug 3, 2020
Merged

[SYCL] Test for fix of linked alloca's deps #1470

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
d8297bd
[SYCL] Fix linked AllocaCmd dependencies
Apr 3, 2020
0d10b3d
Merge branch 'sycl' into private/s-kanaev/fix-linked-allocacmd-deps
Apr 8, 2020
5a44580
[SYCL] Add test
Apr 8, 2020
49a8909
[SYCL] Fix test
Apr 9, 2020
462ee07
[SYCL] Fix test
Apr 9, 2020
c8960d8
Merge branch 'sycl' into private/s-kanaev/fix-linked-allocacmd-deps
Jun 11, 2020
704fdb0
[SYCL] Add unit-test
Jun 17, 2020
0abc928
[SYCL] Fix lit test
Jun 17, 2020
e84cad7
[SYCL] Fix style issue
Jun 17, 2020
fa0dda4
[SYCL] Fix style issue
Jun 17, 2020
658b057
[SYCL] Fix lit test
Jun 17, 2020
107fe7f
[SYCL] Apply suggestion
Jun 17, 2020
83e7688
Merge branch 'sycl' into private/s-kanaev/fix-linked-allocacmd-deps
Jul 30, 2020
897b1b1
[SYCL] Remove redundant test
Jul 30, 2020
b85c3df
Merge remote-tracking branch 'public/sycl' into private/s-kanaev/fix-…
Jul 30, 2020
ba8a54d
[SYCL] Fix testing
Jul 30, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions sycl/source/detail/scheduler/graph_builder.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -618,6 +618,15 @@ AllocaCommandBase *Scheduler::GraphBuilder::getOrCreateAllocaForReq(
} else {
LinkedAllocaCmd->MIsActive = false;
Record->MCurContext = Queue->getContextImplPtr();

std::set<Command *> Deps =
findDepsForReq(Record, Req, Queue->getContextImplPtr());
for (Command *Dep : Deps) {
AllocaCmd->addDep(DepDesc{Dep, Req, AllocaCmd});
Dep->addUser(AllocaCmd);
}
updateLeaves(Deps, Record, Req->MAccessMode);
addNodeToLeaves(Record, AllocaCmd, Req->MAccessMode);
}
}
}
Expand Down