Skip to content

[0029] Updates to Bound's checking behavior text: BC done at matrix/vector granularity. - #567

Open
Anupama Chandrasekhar (anupamachandra) wants to merge 2 commits into
microsoft:mainfrom
anupamachandra:anupamac/clarify-oob-behavior-2
Open

[0029] Updates to Bound's checking behavior text: BC done at matrix/vector granularity.#567
Anupama Chandrasekhar (anupamachandra) wants to merge 2 commits into
microsoft:mainfrom
anupamachandra:anupamac/clarify-oob-behavior-2

Conversation

@anupamachandra

Copy link
Copy Markdown
Contributor

Updates per feedback in #558

@damyanp Damyan Pepper (damyanp) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - we should make sure we get at least one approval from a domain expert (ie not me) before merging this.

Before merging, can you update the PR title / description to be more about the substance of the change rather than it being in response to Tex's feedback please?

@anupamachandra Anupama Chandrasekhar (anupamachandra) changed the title [0029] Updates to Bound's checking behavior per Tex's feedback [0029] Updates to Bound's checking behavior text: BC done at matrix/vector granularity. Jul 16, 2025
@jenatali

Copy link
Copy Markdown
Member

If any part of the matrix load is out of bounds then the entire matrix load will return zero.

That sounds like additional test content that needs to be added. Otherwise we should relax the wording to indicate that implementations may return zero from the entire matrix load.

@V-FEXrt

Copy link
Copy Markdown
Collaborator

Jesse Natalie (@jenatali) IIRC will/must is the correct goal here. I believe there was a request for consistent behavior on OOB access

@jenatali

Copy link
Copy Markdown
Member

Jesse Natalie (@jenatali) IIRC will/must is the correct goal here. I believe there was a request for consistent behavior on OOB access

Got it. Will be a pain to do in WARP but oh well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

7 participants