-
Notifications
You must be signed in to change notification settings - Fork 31
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
Tile configuration addition pass #890
Conversation
68fba75
to
b6137c5
Compare
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.
You really need tests.
b6137c5
to
951f7e5
Compare
@KavithaTipturMadhu On a second thought on the sames and what might have created also confusion on @adam-smnk side: l suggest renaming TileConfig to IntelAMXTileConfig in both pass names, etc. Then it;s clear this has nothing to do with tiling in general but with Intel AMX in particular. |
c106d68
to
49bbe8f
Compare
49bbe8f
to
a6134d6
Compare
Test cases to be added, draft PR in progress. Runtime and code changes are done, licm hoisting is incomplete, working on it.
Test cases to be added, draft PR in progress. Runtime and code changes are done, licm hoisting is incomplete, working on it.