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.

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

target/ppc: Drop VECTOR_FOR_INORDER_I #95

Open
wants to merge 6 commits into
base: ferst-tcg-config_128
Choose a base branch
from

Conversation

mferst
Copy link

@mferst mferst commented Jun 6, 2022

No description provided.

mferst added 6 commits May 30, 2022 13:46
Move vpmsum[bhw] to decodetree and drop the use of VECTOR_FOR_INORDER_I
in helpers, since there is no need to access the elements in any
particular order.
We just need to operate over each element, no need for any particular
order.
We just need to operate over each element, no need for any particular
order.
We just need to operate over each element, no need for any particular
order. Also, use Vsr[BD] instead of the VBPERM[DQ]_INDEX macros, they
are essentially the same thing.
The helper does not operate in place, elements already processed are not
reused in the calculations, and the index is only used to operate over
each element, so there is no order requirement.
As in vpkpx, the helpers do not operate in place, elements already
processed are not reused in the calculations, and the index is only
used to operate over each element, so the use of VECTOR_FOR_INORDER_I is
not required. This macro is also removed, as this is its last use.
@mferst mferst self-assigned this Jun 6, 2022
Copy link

@alqotel alqotel left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link

@luporl luporl left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants