Make GPUInterpreter extensible #634
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
3 times, most recently
from
64e5353
to
55440bb
Compare
vchuravy
force-pushed
the
vc/plugins
branch
from
dfd5c35
to
e9d1372
Compare
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
2 times, most recently
from
f647ad6
to
2e306cb
Compare
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
from
2e306cb
to
53c94a8
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## vc/plugins #634 +/- ##
==============================================
- Coverage 73.68% 72.29% -1.40%
==============================================
Files 24 24
Lines 3310 3353 +43
==============================================
- Hits 2439 2424 -15
- Misses 871 929 +58 ☔ View full report in Codecov by Sentry. |
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
2 times, most recently
from
4e127d6
to
6f8532b
Compare
vchuravy
force-pushed
the
vc/plugins
branch
from
e9d1372
to
828ee63
Compare
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
5 times, most recently
from
9af6e00
to
9fa52ca
Compare
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
from
9fa52ca
to
052a118
Compare
Draft
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
from
052a118
to
4ef6019
Compare
Currently Enzyme uses it's own AbstractInterpreter, in particular to handle inlining blocking of functions with custom rules and to handle nested autodiff operations. - [ ] Create a version of Enzyme with this - [ ] Support a version of `gpuc.deferred(meta)`
vchuravy
force-pushed
the
09-26-make_gpuinterpreter_extensible
branch
from
4ef6019
to
1638cc2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently Enzyme uses it's own AbstractInterpreter, in particular to
handle inlining blocking of functions with custom rules and to handle
nested autodiff operations.
gpuc.deferred(meta)