Skip to content

[mlir] Skip uniquer locks when a context starts without threading (#2… - #5113

Merged
ronlieb merged 4 commits into
amd-stagingfrom
amd/dev/rlieberm/reland/MLIRPath
Oct 26, 2026
Merged

ronlieb merged 4 commits into
amd-stagingfrom
amd/dev/rlieberm/reland/MLIRPath

Conversation

@ronlieb

@ronlieb ronlieb commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator

…35294)

attempt to land, has mysterious MLIR LIT failure

@ronlieb
ronlieb requested a review from aobolensk October 25, 2026 00:48
aobolensk added a commit to aobolensk/llvm-project that referenced this pull request Oct 26, 2026
amd-staging disables MLIR threading for every context, so its uniquers skip locks and this test races

Fixes the check-mlir failure in ROCm#5113
aobolensk added a commit to aobolensk/llvm-project that referenced this pull request Oct 26, 2026
MLIR_ENABLE_THREADS is never defined, which left every context without threading
and its uniquers lock-free. Fixes the MLIRIRTests failure in
ROCm#5113

@aobolensk aobolensk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fix should help landing this: #5124

ronlieb pushed a commit that referenced this pull request Oct 26, 2026
MLIR_ENABLE_THREADS is never defined, which left every context without
threading and its uniquers lock-free

Fixes the MLIRIRTests failure in
#5113
@ronlieb
ronlieb enabled auto-merge (squash) October 26, 2026 16:08
@ronlieb
ronlieb merged commit ef84404 into amd-staging Oct 26, 2026
170 of 174 checks passed
@ronlieb
ronlieb deleted the amd/dev/rlieberm/reland/MLIRPath branch October 26, 2026 19:37
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