Skip to content

Conversation

@jungpark-mlir
Copy link
Owner

New contributor declaration

  • I am not making a trivial change, such as fixing a typo in a comment.

  • I have written a PR description following these
    rules.

  • I have run pre-commit run --from-ref origin/main --to-ref HEAD.

  • Select one of the following.

    • I have added tests.
      • /test for lit tests
      • /unittest for C++ tests
      • /python/test for end-to-end tests
    • This PR does not need a test because FILL THIS IN.
  • Select one of the following.

    • I have not added any lit tests.
    • The lit tests I have added follow these best practices,
      including the "tests should be minimal" section. (Usually running Python code
      and using the instructions it generates is not minimal.)

partitioning of the code into stages.
backup
can correctly insert fence.
update interfaces per recent changes
make it work actually
fix wrongly offset insertion
refactor loop
code cleanup
barrier should be inserted from the warp causing the dependency.
Added builtin split_warp_pipeline(), inserting the builtin
splits the code region into two pipeline clusters.
now runs on mi350
- polish conversion code
- found an important fix needed, just commented for now.
@jungpark-mlir jungpark-mlir merged commit f298346 into newpp2 Oct 27, 2025
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