-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Make tests/ui/abi/debug.rs cross-compile #143464
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
Make tests/ui/abi/debug.rs cross-compile #143464
Conversation
This comment has been minimized.
This comment has been minimized.
2f41577
to
64bec0f
Compare
Two lines. |
@workingjubilee As a newcomer to the Rust community and related work, I realize our PR discussions might have contained some miscommunications – I sincerely appreciate your time and patience throughout. Thank you especially for your revisions to the PR! |
@YingkaiLi-VM If you are happy with this PR as it is then you need do nothing. My apologies again for the confusion. |
@bors r+ rollup |
Rollup of 11 pull requests Successful merges: - #143130 (doc(std): clarify `NonZero<T>` usage limitation in doc comment) - #143415 (Get rid of build-powerpc64le-toolchain.sh) - #143464 (Make tests/ui/abi/debug.rs cross-compile) - #143482 (Fix short linker error output) - #143524 (Move `stable_mir` back to its own crate) - #143528 (interpret: rename StackPopCleanup) - #143551 (Dont resolve instance of root in `mir_callgraph_cyclic`) - #143558 (mbe: Refactors and function extractions in `compile_declarative_macro`) - #143563 (std: fix typo in `std::path`) - #143564 (compiler: Deduplicate `must_emit_unwind_tables()` comments) - #143577 (Disable download-rustc for library profile) r? `@ghost` `@rustbot` modify labels: rollup
Resumption of #141979, making the test cross-compile.
@YingkaiLi-VM My apologies if this is undesired. 64bec0f was the two lines of difference needed. Feel free to reopen a new version of your PR if you would rather rebase and fix it up again.