Skip to content

Conversation

mshockwave
Copy link
Member

Both RISC-V and AMDGPU(GCN) deploy two VirtRegRewriter in their codegen pipeline. This test prematurely stops at the first one, which doesn't cleanup the virtual register map and cause an assertion failure. Ideally we can solve this by teaching -stop-after how to stop at the last instance of a Pass, but we're just marking XFAIL for these two targets for now.

…or RISCV and AMDGPU

Both RISC-V and AMDGPU(GCN) deploy two VirtRegRewriter in their codegen
pipeline. This test prematurely stops at the first one, which doesn't
cleanup the virtual register map and cause an assertion failure.
Ideally we can solve this by teaching `-stop-after` how to stop at the last
instance of a Pass, but we're just marking XFAIL for these two targets for now.
Copy link
Collaborator

@topperc topperc left a comment

Choose a reason for hiding this comment

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

LGTM

@mshockwave mshockwave merged commit 03be448 into llvm:main Jan 11, 2024
@mshockwave mshockwave deleted the patch/xfail-dbg-test branch January 11, 2024 00:47
; Both RISC-V and AMDGPU(GCN) deploy two VirtRegRewriter in their codegen
; pipeline. This test prematurely stops at the first one, which doesn't cleanup
; the virtual register map and cause an assertion failure. Ideally we can solve
; this by teaching `-stop-after` how to stop at the last instance of a Pass,
Copy link
Contributor

Choose a reason for hiding this comment

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

You can use -stop-after=passname,1 to skip to the next instance

justinfargnoli pushed a commit to justinfargnoli/llvm-project that referenced this pull request Jan 28, 2024
…or RISCV and AMDGPU (llvm#77631)

Both RISC-V and AMDGPU(GCN) deploy two VirtRegRewriter in their codegen
pipeline. This test prematurely stops at the first one, which doesn't
cleanup the virtual register map and cause an assertion failure. Ideally
we can solve this by teaching `-stop-after` how to stop at the last
instance of a Pass, but we're just marking XFAIL for these two targets
for now.
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.

3 participants