Skip to content

fix the Shared layout in OptimizeDotOperands pass to generate correct swizzling code#2180

Merged
ptillet merged 2 commits intotriton-lang:mainfrom
BinFan:binf/transp-bug-1937
Sep 13, 2023
Merged

fix the Shared layout in OptimizeDotOperands pass to generate correct swizzling code#2180
ptillet merged 2 commits intotriton-lang:mainfrom
BinFan:binf/transp-bug-1937

Conversation

@BinFan
Copy link
Copy Markdown
Contributor

@BinFan BinFan commented Aug 25, 2023

fix bug #1937

@BinFan BinFan requested a review from ptillet as a code owner August 25, 2023 03:10
@BinFan BinFan marked this pull request as draft August 25, 2023 16:04
@BinFan
Copy link
Copy Markdown
Contributor Author

BinFan commented Aug 25, 2023

converting to draft, found some issue in some tests

… swizzling code

Also some addtional fix in Pipeline pass

This fix bug triton-lang#1937
@BinFan BinFan force-pushed the binf/transp-bug-1937 branch from 2b5f369 to 020907b Compare August 25, 2023 21:00
@BinFan
Copy link
Copy Markdown
Contributor Author

BinFan commented Aug 25, 2023

added some additional fix, remove draft status

@BinFan BinFan closed this Aug 25, 2023
@BinFan BinFan reopened this Aug 25, 2023
@BinFan BinFan marked this pull request as ready for review August 25, 2023 21:05
@ptillet
Copy link
Copy Markdown
Collaborator

ptillet commented Sep 13, 2023

This looks good to me! Sorry for the delay reviewing this

@ptillet ptillet merged commit 38a2ecd into triton-lang:main Sep 13, 2023
alexander-zinoviev pushed a commit to alexander-zinoviev/triton that referenced this pull request Sep 21, 2023
… correct swizzling code (triton-lang#2180)

fix bug triton-lang#1937

Co-authored-by: Philippe Tillet <phil@openai.com>
pingzhuu pushed a commit to siliconflow/triton that referenced this pull request Apr 2, 2024
… correct swizzling code (triton-lang#2180)

fix bug triton-lang#1937

Co-authored-by: Philippe Tillet <phil@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants