Merge back to back slices on the same dim#19128
Merge back to back slices on the same dim#19128meta-codesync[bot] merged 1 commit intopytorch:mainfrom
Conversation
Summary: If we have back to back slices on the same dimension, we can remove the top slice and just perform the second one. RemoveOrReplacePassInterface will handle a whole cascade if it exists. Handles negative start/end indices by canonicalizing them, and negative dims by normalizing against ndim. Clamps the merged end to the parent's end to handle child ranges that exceed the parent output size. Differential Revision: D102425537
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/19128
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 2 Cancelled Jobs, 2 Unrelated FailuresAs of commit 0204b9e with merge base 2330652 ( CANCELLED JOBS - The following jobs were cancelled. Please retry:
BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@DrJessop has exported this pull request. If you are a Meta employee, you can view the originating Diff in D102425537. |
This PR needs a
|
Summary:
If we have back to back slices on the same dimension, we can remove the top slice and just perform the second one. RemoveOrReplacePassInterface will handle a whole cascade if it exists.
Handles negative start/end indices by canonicalizing them, and negative dims by normalizing against ndim. Clamps the merged end to the parent's end to handle child ranges that exceed the parent output size.
Differential Revision: D102425537