-
Notifications
You must be signed in to change notification settings - Fork 79
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.
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
[WIP]: Proof of concept going from jax to riscv #3674
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3674 +/- ##
=======================================
Coverage 91.29% 91.30%
=======================================
Files 466 467 +1
Lines 58357 58410 +53
Branches 5624 5631 +7
=======================================
+ Hits 53278 53331 +53
Misses 3629 3629
Partials 1450 1450 ☔ View full report in Codecov by Sentry. |
donated_inputs = [ | ||
inp | ||
for inp, attr in zip(func_op.args, func_op.arg_attrs, strict=True) | ||
if isinstance(inp.type, TensorType) and "tf.aliasing_output" in attr.data | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's use a tuple for this, and bail out if this is empty?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, we pop below...
No description provided.