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

Omit the where predicate of pad when found safe to do so #3708

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

naoyam
Copy link
Collaborator

@naoyam naoyam commented Jan 14, 2025

No description provided.

@naoyam
Copy link
Collaborator Author

naoyam commented Jan 14, 2025

!test

@naoyam naoyam mentioned this pull request Jan 14, 2025
@naoyam naoyam added the rope label Jan 14, 2025
naoyam and others added 4 commits January 14, 2025 14:06
Currently only has one parameter. Also added some minor tweaks. 

- Previously gridDim.x was static, which is now symbolic. 
- Rejects transpose-like patterns for now as they would need scheduling
like what the transpose does.
@naoyam naoyam force-pushed the omit_pad_predicate branch from 9ed94ec to d6cef9a Compare January 14, 2025 22:06
@naoyam naoyam changed the base branch from main to idmodel_indexing_minor_refactoring January 14, 2025 22:07
naoyam added a commit that referenced this pull request Jan 14, 2025
Just small minor refactoring to add `TensorIndexer::getIndexingPath`,
which is required in #3708. Nothing should change with this PR itself.
Base automatically changed from idmodel_indexing_minor_refactoring to main January 14, 2025 22:10
@naoyam
Copy link
Collaborator Author

naoyam commented Jan 14, 2025

!test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants