Skip to content

Conversation

@ysiraichi
Copy link
Collaborator

This PR refactors AsStrided node implementation, and improves its error handling by making it override SafeLower.

Key Changes:

  • Replaced Lower override by SafeLower
  • Inlined LowerAsStrided into the SafeLower implementation + comments
  • Added check CheckSpecFitsInput() at tracing and lowering time
  • Moved GetArrayStridePermutation() out of AsStrided class, and renamed it to GetDescendingOrderPermutation()
    • Returns a descending order permutation of the given span
  • Moved StrideIsSupported out of AsStrided class, and renamed it to AsStridedIsSupported()

@ysiraichi ysiraichi marked this pull request as draft November 26, 2025 19:49
@ysiraichi
Copy link
Collaborator Author

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