Skip to content

Fix #697 - #698

Closed
jonathanpallant wants to merge 1 commit into
masterfrom
remove-nomem
Closed

Fix #697#698
jonathanpallant wants to merge 1 commit into
masterfrom
remove-nomem

Conversation

@jonathanpallant

@jonathanpallant jonathanpallant commented Oct 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  1. Put fences before and after the intrinsics (it would be bad if the disable interrupt operation was moved before the primask read).
  2. Remove the nomem attribute, so that LLVM believes the asm blocks touch memory and hence cannot move across the fences.

@jonathanpallant
Fix #697
23b8ae5 
1. Put fences before and after the intrinsics (it would be bad if the
   disable interrupt operation was moved before the primask read).
2. Remove the `nomem` attribute, so that LLVM believes the asm blocks
   touch memory and hence cannot move across the fences.
@jordens jordens mentioned this pull request Oct 17, 2026
Merged
@jonathanpallant

jonathanpallant commented Oct 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Superceded by #699

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@diondokter diondokter diondokter approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jonathanpallant @diondokter