-
Notifications
You must be signed in to change notification settings - Fork 347
Add krnl prefetch #2786
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
Add krnl prefetch #2786
Conversation
… pass Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
Signed-off-by: Alexandre Eichenberger <[email protected]>
cleaned up old dialect builder for affine prefetch that are not in use. |
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.
LGTM.
// Code lifted from affine prefetch as is. | ||
// I have seen parsing errors when multiple '#x' are used in the indices, could | ||
// not tell why. | ||
// krnl.prefetch %arg0[%1#0, %1#1, %3], read, locality<3>, 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.
How about affine.prefetch %arg0[%1#0, %1#1, %3], read, locality<3>, data
?
Got included as part of other PRs |
Cannot add
affine.prefetch
during ONNX to KRNL lowering because affine prefetch check the enclosing loop nests, which are not yetaffine.for
loops during KRNL lowering.Thus created a placeholder
krnl.prefetch
that will later be lowered toaffine.prefetch
.