[Mlir-commits] [mlir] [MLIR] Determine contiguousness of memrefs with dynamic dimensions (PR #142421)
Andrzej Warzyński
llvmlistbot at llvm.org
Wed Jun 18 01:27:48 PDT 2025
banach-space wrote:
> Just a drive-by comment - when I started contributing here, a reviewer kindly pointed out to me in this [#135096 (comment)](https://github.com/llvm/llvm-project/pull/135096#issuecomment-2797902624) that force pushing isn't recommended https://llvm.org/docs/GitHub.html#rebasing-pull-requests-and-force-pushes
+1
That said, that document also states:
> Sometimes, a rebase might be needed to update your branch with a fix for a test or in some dependent code.
I do occasionally rebase to keep my branch up-to-date with main, especially for longer-lived PRs that are still under review. I'm always happy to adapt that habit, of course!
For me, the real concern is around “rebase + squash”, since that can wipe out intermediate commits that were already reviewed or discussed - and that history can be valuable.
Just posting this as a side note to clarify my current workflow - again, very open to adjusting it if needed.
https://github.com/llvm/llvm-project/pull/142421
More information about the Mlir-commits
mailing list