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

Post-increment load/store in trace parser script #112

Open
2 tasks
sermazz opened this issue Aug 9, 2024 · 0 comments
Open
2 tasks

Post-increment load/store in trace parser script #112

sermazz opened this issue Aug 9, 2024 · 0 comments

Comments

@sermazz
Copy link
Collaborator

sermazz commented Aug 9, 2024

Posting here as a to-do.
The trace parser script implicitly assumes that all load and store instructions do not use post-increment addressing.
https://github.com/pulp-platform/mempool/blob/main/hardware/scripts/gen_trace.py#L189-L199
extras['alu_result'] is always printed as accessed address, but Reg[rs1] is the actually accessed address when post-increment is used.
To fix:

  • also trace i_snitch.is_postincr in the tracer in mempool_cc
  • enhance the lines above of the tracer script to change the printed memory address of post-increment instructions
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

No branches or pull requests

1 participant