Minor docs update #2075
Open
Minor docs update #2075
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jan-wassenberg
requested changes
Apr 10, 2024
| type than those in `V`. The number of lanes in `V` and `VI` may differ, e.g. | ||
| a full-length table vector loaded via `LoadDup128`, plus partial vector `VI` | ||
| of 4-bit indices. | ||
| type than those in `V` and are loaded uses the standard load instructions. |
| @@ -1928,16 +1928,16 @@ their operands into independently processed 128-bit *blocks*. | |||
|
|
|||
| All other ops in this section are only available if `HWY_TARGET != HWY_SCALAR`: | |||
|
|
|||
| * `V`: `{u,i}` \ | |||
| * `V`, `VI`: `{u,i}` \ | |||
| <code>VI **TableLookupBytes**(V bytes, VI indices)</code>: returns | |||
| `bytes[indices[i]]`. Uses byte lanes regardless of the actual vector types. | |||
| Results are implementation-defined if `indices[i] < 0` or `indices[i] >= | |||
| HWY_MIN(Lanes(DFromV<V>()), 16)`. `VI` are integers, possibly of a different | |||
There was a problem hiding this comment.
How about something like "VI are normal vectors with integer lane type possibly differing from the lanes of V. Note that TableLookupLanes has an unspecified, potentially non-vector type type for the indices."?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add data types for
SetTableIndicesand made it clearer on how to load indices forTableLookUpBytes