Conversation
|
Do we need to update llvm version of IMEX as well? |
Yes. Thanks for remindering. |
If so, please also raise a PR in this forked IMEX repo and request review of @dchigarev. |
Just to remind: we have a staging branch in the main imex repository, so all PRs should be submitted there. One more thing: from our previous experience updating LLVM version for IMEX may not be that easy as simply updating llvm-commit hash. Given that GPU pipeline in GC works fine even with old LLVM for IMEX, we can leave it as it is for now and update it in a separate PR (if it would appear too complicated to make it now) |
e268bed to
d8662c0
Compare
|
@dchigarev Hi. The current updated llvm commit has caused page fault runtime error in gpu The detailed CI log is accessible at: https://github.com/intel/graph-compiler/actions/runs/11043869278/job/30678842932?pr=357. |
Signed-off-by: dchigarev <dmitry.chigarev@intel.com>
The |
Thanks a lot for help fixing this! |
Update llvm commit to involve the fix for unpack op canonicalization and
isaConvolutionOpInterface.PR #101 requires this update to pass CI check.