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

i#3544 RV64: Add the missing architecture-related code to drcachesim #6898

Merged
merged 3 commits into from
Jul 22, 2024

Conversation

chenhy0106
Copy link
Contributor

@chenhy0106 chenhy0106 commented Jul 21, 2024

Fix the issue in drcachesim where missing architecture-related code leads to incorrect counting results.

Issue: #3544

Fix the issue in insert_save_type_and_size where the absence of code related to the RISC-V 64 architecture causes problems with counting instruction fetches.

i#3544
@chenhy0106 chenhy0106 changed the title i#3544 RV64: Add the missing RV64 architecture-related code to drcachesim i#3544 RV64: Add the missing architecture-related code to drcachesim Jul 21, 2024
@chenhy0106 chenhy0106 requested review from derekbruening and ksco July 21, 2024 14:46
clients/drcachesim/tracer/instru_online.cpp Outdated Show resolved Hide resolved
clients/drcachesim/tracer/tracer.cpp Outdated Show resolved Hide resolved
@ksco ksco merged commit fe2e442 into DynamoRIO:master Jul 22, 2024
16 of 17 checks passed
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