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

[PowerPC][Atomics] Remove redundant block to clear reservation #68430

Merged
merged 2 commits into from
Oct 13, 2023

Conversation

bzEq
Copy link
Collaborator

@bzEq bzEq commented Oct 6, 2023

This PR is following what https://reviews.llvm.org/D134783 does for quardword CAS.

@bzEq bzEq self-assigned this Oct 6, 2023
@bzEq bzEq requested a review from stefanp-ibm October 6, 2023 17:11
@bzEq bzEq changed the title [PowerPC] Remove redundant block to clear reservation [PowerPC][Atomics] Remove redundant block to clear reservation Oct 7, 2023
Copy link
Member

@nemanjai nemanjai left a comment

Choose a reason for hiding this comment

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

LGTM.

@nemanjai
Copy link
Member

The failing RISCV test case reported by buildkite definitely seems unrelated.

@bzEq bzEq merged commit 3104681 into llvm:main Oct 13, 2023
@bzEq bzEq deleted the adjust-cas-codegen branch October 13, 2023 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants