Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add an allow_negative_indices option to lax.dynamic_slice and lax.dynamic_update_slice. #24931

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add an allow_negative_indices option to lax.dynamic_slice and lax.dynamic_update_slice. #24931

wants to merge 1 commit into from

Commits on Nov 16, 2024

  1. Add an allow_negative_indices option to lax.dynamic_slice and lax.dyn… 

    …amic_update_slice.

The goal of this change is to avoid generating code to wrap negative indices back into range in cases where we know it doesn't matter. Change scan to pass allow_negative_indices=True to avoid emitting index wrapping code for each scan argument.

PiperOrigin-RevId: 697210115
    @hawkinsp @Google-ML-Automation
    hawkinsp authored and Google-ML-Automation committed Nov 16, 2024
    Configuration menu
    Copy the full SHA
    7a1899a View commit details
    Browse the repository at this point in the history