Skip to content

Conversation

govardhnn
Copy link
Contributor

No description provided.

COMPRESS,
WHOLE_REG_MOVE,
UNKNOWN,
NONE
Copy link
Contributor

Choose a reason for hiding this comment

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

NONE should be before UNKNOWN.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modified.

@kathlenemagnus-mips
Copy link
Contributor

@govardhnn your design doc is missing the critical information that I'm looking for which is "what uops are generated for each type of permutation instruction?". It would be great to include a simple example of uop generation for each type of uop generator that you plan to add.

@govardhnn
Copy link
Contributor Author

govardhnn commented Jun 2, 2025 via email

@klingaard
Copy link
Collaborator

Hey @govardhnn where are you with this PR? Specifically, did you address @kathlenemagnus requests? Also, can you get regression to pass again?

@govardhnn
Copy link
Contributor Author

govardhnn commented Jul 19, 2025

Hi @klingaard and @kathlenemagnus-mips I have been on a personal break for a while - I should have informed the team earlier, apologies..
Can I get back to this by the end of the month? Or please feel free to take this over if the feature is on the critical path.

Thanks,

@klingaard
Copy link
Collaborator

Hey @govardhnn, yes, please feel free to take the time you need. We do appreciate the contributions that folks make to the model.

Copy link
Collaborator

@klingaard klingaard left a comment

Choose a reason for hiding this comment

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

Preemptive acceptance.

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