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.

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

Partially fuse loops in addition #56

Open
drhagen opened this issue Feb 16, 2024 · 0 comments
Open

Partially fuse loops in addition #56

drhagen opened this issue Feb 16, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@drhagen
Copy link
Owner

drhagen commented Feb 16, 2024

Right now, loops inside a Sum node are supposed to be fused if they start with same index. However, this only happens if they are adjacent. If a loop with a different index is placed between two fusable loops in the Sum, they are not fused. This is both inefficient and causes a crash (because the variables are redeclared).

Fused:

tensora 'a(i) = b(i) + c(i) + d(j,i)'

Not fused:

tensora 'a(i) = b(i) + c(j,i) + d(i)'
@drhagen drhagen added the bug Something isn't working label Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant