Skip to content

Fixing vnsrl[a] operation for LMUL < 1#131

Open
Joao-Pedro-Cabral wants to merge 2 commits intovproc:mainfrom
Joao-Pedro-Cabral:vnsrla
Open

Fixing vnsrl[a] operation for LMUL < 1#131
Joao-Pedro-Cabral wants to merge 2 commits intovproc:mainfrom
Joao-Pedro-Cabral:vnsrla

Conversation

@Joao-Pedro-Cabral
Copy link

There are several bugs for this corner case:

  • Instruction only ends after “two vreg cycles”;
  • Instruction writes in the upper half of the result buffer;

Solution:

  • Register Pack write at the lower half of the result buffer
  • Processor Pipeline ends the instruction after “one vreg cycles”;

Depends on: #128.

@ParkerJones567
Copy link
Contributor

This issue might still be present in vicuna2.0, please test with the current development branch

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.

2 participants