[0029] Updates to Bound's checking behavior text: BC done at matrix/vector granularity.#567
Conversation
damyanp
left a comment
There was a problem hiding this comment.
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?
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. |
|
@jenatali IIRC |
Got it. Will be a pain to do in WARP but oh well. |
Updates per feedback in #558