Skip to content

Conversation

erwei-xilinx
Copy link
Collaborator

No description provided.

@erwei-xilinx
Copy link
Collaborator Author

Hi @andrej, one of the test for bd-chains-and-dma-tasks was disabled after llvm bump, due to changes in the canonicalizer behavior that the last block is now canonicalized away.

@erwei-xilinx erwei-xilinx marked this pull request as ready for review September 5, 2025 22:09
@erwei-xilinx
Copy link
Collaborator Author

Hi @andrej, one of the test for bd-chains-and-dma-tasks was disabled after llvm bump, due to changes in the canonicalizer behavior that the last block is now canonicalized away.

So I think this error message shall never get triggered, as any disconnected block shall get canonicalized away, before triggering this error message.

@erwei-xilinx erwei-xilinx added this pull request to the merge queue Sep 5, 2025
@andrej
Copy link
Collaborator

andrej commented Sep 5, 2025

That makes sense. Thanks for the update!

Merged via the queue into Xilinx:main with commit 755411a Sep 5, 2025
53 checks passed
@erwei-xilinx erwei-xilinx deleted the bump_llvm_064f02da branch September 5, 2025 22:54
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