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: Fixed atomic functions and macros #6089

Merged
merged 5 commits into from
Jun 9, 2023

Conversation

ksco
Copy link
Contributor

@ksco ksco commented May 27, 2023

This patch fixed some atomic functions and macros of the RV64 port.

Issue: #3544

@ksco ksco requested a review from abhinav92003 May 27, 2023 09:07
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
@abhinav92003
Copy link
Contributor

Please request re-review from the UI when ready.

core/arch/atomic_exports.h Outdated Show resolved Hide resolved
core/arch/atomic_exports.h Outdated Show resolved Hide resolved
@ksco ksco requested a review from derekbruening June 9, 2023 03:06
Copy link
Contributor

@derekbruening derekbruening left a comment

Choose a reason for hiding this comment

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

(I didn't go through the exact sequences)

@ksco ksco added this pull request to the merge queue Jun 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 9, 2023
@derekbruening derekbruening merged commit 4737397 into DynamoRIO:master Jun 9, 2023
@ksco ksco deleted the atomic branch June 23, 2023 18:18
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