Skip to content

[AMD] Disable LLVM vector combine pass#9260

Merged
antiagainst merged 2 commits into
triton-lang:mainfrom
antiagainst:amd-vector-combine
May 12, 2026
Merged

[AMD] Disable LLVM vector combine pass#9260
antiagainst merged 2 commits into
triton-lang:mainfrom
antiagainst:amd-vector-combine

Conversation

@antiagainst
Copy link
Copy Markdown
Member

@antiagainst antiagainst commented Jan 21, 2026

This pass is causing more difficulties than help
for us.

@antiagainst antiagainst force-pushed the amd-vector-combine branch 2 times, most recently from 9b81876 to 0109f08 Compare February 18, 2026 04:37
@antiagainst antiagainst force-pushed the amd-vector-combine branch from 84bf34b to c67b24a Compare May 8, 2026 21:59
@ThomasRaoux
Copy link
Copy Markdown
Collaborator

if it is just for debugging, would using the existing DISABLE_LLVM_OPT work?

@antiagainst
Copy link
Copy Markdown
Member Author

We tested internally on cases it's not causing regressions. So I do want to disable it permantly. The introduced knob is more for debugging just in case some bad cases surface up after this. I can also remove it.

@ThomasRaoux
Copy link
Copy Markdown
Collaborator

We tested internally on cases it's not causing regressions. So I do want to disable it permantly. The introduced knob is more for debugging just in case some bad cases surface up after this. I can also remove it.

and doing DISABLE_LLVM_OPT=disable-vector-combine doesn't work?

@antiagainst antiagainst changed the title [AMD] Add a knob to disable vector combine pass [AMD] Disable LLVM vector combine pass May 8, 2026
@antiagainst
Copy link
Copy Markdown
Member Author

Yeah that works for debugging. I think the new option was confusing so I dropped it.

@antiagainst antiagainst marked this pull request as ready for review May 8, 2026 22:26
@antiagainst antiagainst merged commit 36394d4 into triton-lang:main May 12, 2026
9 checks passed
@antiagainst antiagainst deleted the amd-vector-combine branch May 12, 2026 03:57
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