fix: omit `nomem` option to prevent compiler reorderings #11

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.

Sign up for GitHub

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

Jump to bottom

Open

tsoutsman wants to merge 1 commit into theseus-os:master from tsoutsman:fix-fences

Member

tsoutsman commented Oct 12, 2023

See mkroening/interrupt-ref-cell#5 (comment).

Also adds the preserves_flags option to x86_64 assembly as the interrupt flag doesn't need to be preserved.


          fix: omit nomem option to prevent compiler reorderings

cdf3ee2

See mkroening/interrupt-ref-cell#5 (comment).

Also adds the `preserves_flags` option to `x86_64` assembly as the
interrupt flag [doesn't need to be preserved][rust-ref].

[rust-ref]: https://doc.rust-lang.org/reference/inline-assembly.html#rules-for-inline-assembly

Signed-off-by: Klimenty Tsoutsman <[email protected]>

Member

kevinaboos commented Oct 12, 2023

Thanks, will wait on the x86_64 PR here before merging: rust-osdev/x86_64#440

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

Labels

None yet