Skip to content
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

Jump to bottom

Unify the llvm context of host and device #7251

Closed
lin-hitonami opened this issue Jan 29, 2023 · 0 comments · Fixed by #7249
Closed

Unify the llvm context of host and device #7251

lin-hitonami opened this issue Jan 29, 2023 · 0 comments · Fixed by #7249

Comments

@lin-hitonami
Copy link
Contributor

Describe the bug
The llvm_context_host_ and llvm_context_device_ are never both used in the same backend, so we can unify them and only use a single llvm_context_.
@github-project-automation github-project-automation bot moved this to Untriaged in Taichi Lang Jan 29, 2023
@lin-hitonami lin-hitonami mentioned this issue Jan 29, 2023
Merged
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Taichi Lang Feb 1, 2023
lin-hitonami added a commit that referenced this issue Feb 1, 2023
@lin-hitonami
[llvm] Unify the llvm context of host and device (#7249)
2603e66 
Issue: fixes #7251
The` llvm_context_host_` and `llvm_context_device_` are never both used
in the same backend, so we can unify them and only use a single
`llvm_context_`.
### Brief Summary
@lin-hitonami lin-hitonami mentioned this issue Feb 2, 2023
Merged
lin-hitonami added a commit that referenced this issue Feb 2, 2023
@lin-hitonami
[llvm] Remove unused argument 'arch' in LlvmProgramImpl::get_llvm_con…
edfe19f 
…text (#7282)

Issue: #7251

### Brief Summary
quadpixels pushed a commit to quadpixels/taichi that referenced this issue May 13, 2023
@lin-hitonami @quadpixels
[llvm] Unify the llvm context of host and device (taichi-dev#7249)
192e78e 
Issue: fixes taichi-dev#7251
The` llvm_context_host_` and `llvm_context_device_` are never both used
in the same backend, so we can unify them and only use a single
`llvm_context_`.
### Brief Summary
quadpixels pushed a commit to quadpixels/taichi that referenced this issue May 13, 2023
@lin-hitonami @quadpixels
[llvm] Remove unused argument 'arch' in LlvmProgramImpl::get_llvm_con…
af664dc 
…text (taichi-dev#7282)

Issue: taichi-dev#7251

### Brief Summary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Taichi Lang
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[llvm] Unify the llvm context of host and device lin-hitonami/taichi
1 participant
@lin-hitonami