- Notifications
You must be signed in to change notification settings - Fork 13.8k
Share the naked asm impl between cg_ssa and cg_clif #134232
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
Conversation
cc @folkertdev |
r? @chenyukang rustbot has assigned @chenyukang. Use |
This comment has been minimized.
This comment has been minimized.
b8677a2
to d6b3c06
Compare This comment has been minimized.
This comment has been minimized.
d6b3c06
to 0dd4292
Compare Some changes occurred in compiler/rustc_codegen_gcc |
☔ The latest upstream changes (presumably #134381) made this pull request unmergeable. Please resolve the merge conflicts. |
42ae211
to 1358f53
Compare 1358f53
to dcf8b87
Compare r? compiler |
☔ The latest upstream changes (presumably #135959) made this pull request unmergeable. Please resolve the merge conflicts. |
dcf8b87
to 0b43964
Compare ☔ The latest upstream changes (presumably #136332) made this pull request unmergeable. Please resolve the merge conflicts. |
0b43964
to b250776
Compare ☔ The latest upstream changes (presumably #137163) made this pull request unmergeable. Please resolve the merge conflicts. |
b250776
to 1341f83
Compare ef86f9d
to 3066da8
Compare Some changes occurred in compiler/rustc_codegen_ssa |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The cg_clif changes seem fine but I don't know enough about that part to thoroughly review. The rest lgmt!
@bors r+ |
…esleywiser Share the naked asm impl between cg_ssa and cg_clif This was introduced in rust-lang#128004.
Rollup of 8 pull requests Successful merges: - rust-lang#134232 (Share the naked asm impl between cg_ssa and cg_clif) - rust-lang#140312 (Improve pretty-printing of braces) - rust-lang#140437 (enable msa feature for mips in codegen tests) - rust-lang#140438 (Add `rust.debug-assertions-tools` option) - rust-lang#140439 (miri: algebraic intrinsics: bring back float non-determinism) - rust-lang#140445 (Treat ManuallyDrop as ~const Destruct) - rust-lang#140446 (chore: fix some tests) - rust-lang#140448 (Rename `rustc_query_append!` to `rustc_with_all_queries!`) r? `@ghost` `@rustbot` modify labels: rollup
@bors r- Seems to have failed in #140475 (comment) |
@bors try |
Share the naked asm impl between cg_ssa and cg_clif This was introduced in rust-lang#128004. try-job: x86_64-apple-1
☀️ Try build successful - checks-actions |
@bors r=wesleywiser Just a minor cg_clif change since the last approval. |
…iaskrgr Rollup of 9 pull requests Successful merges: - rust-lang#134232 (Share the naked asm impl between cg_ssa and cg_clif) - rust-lang#139624 (Don't allow flattened format_args in const.) - rust-lang#140090 (Check bare function idents for non snake-case name) - rust-lang#140203 (Issue an error when using `no_mangle` on language items) - rust-lang#140450 (ast: Remove token visiting from AST visitor) - rust-lang#140498 (Misc tweaks to HIR typeck (mostly w.r.t. checking calls)) - rust-lang#140504 (transmutability: ensure_sufficient_stack when answering query) - rust-lang#140506 (unstable-book: fix capitalization) - rust-lang#140516 (Replace use of rustc_type_ir by rustc_middle) Failed merges: - rust-lang#140374 (Resolve instance for SymFn in global/naked asm) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#134232 - bjorn3:naked_asm_improvements, r=wesleywiser Share the naked asm impl between cg_ssa and cg_clif This was introduced in rust-lang#128004.
…esleywiser Share the naked asm impl between cg_ssa and cg_clif This was introduced in rust-lang#128004.
…esleywiser Share the naked asm impl between cg_ssa and cg_clif This was introduced in rust-lang#128004.
This was introduced in #128004.