Skip to content

Conversation

YuhanLiin
Copy link
Contributor

The current generated code for MSP430 atomics is not compatible with the rest of the generated code. This PR fixes this.

@YuhanLiin YuhanLiin requested a review from a team as a code owner October 27, 2022 04:32
@rust-highfive
Copy link

r? @reitermarkus

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-tools labels Oct 27, 2022
@burrbull
Copy link
Member

could you add ci tests to prevent this issue in future?

also update changelog

@burrbull
Copy link
Member

@YuhanLiin YuhanLiin closed this Oct 29, 2022
@YuhanLiin YuhanLiin deleted the fix-msp430-atomic branch October 29, 2022 18:02
bors bot added a commit that referenced this pull request Oct 29, 2022
688: Fix generated code for MSP430 atomics r=burrbull a=YuhanLiin

Rebased from #685


Co-authored-by: YuhanLiin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants