Skip to content

Conversation

tgross35
Copy link
Contributor

@tgross35 tgross35 commented Aug 7, 2025

Successful merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

Kivooeo and others added 15 commits August 5, 2025 19:02
This patch fixes the Markdown formatting in
`std::core::iter::Iterator::by_ref`. Code is used inside a link without
the backticks around the code.
these are fixed by polonius=next and polonius=legacy
…859/48001

these are fixed by polonius=legacy, are currently accepted by
polonius=next but won't be by the alpha analysis
`tests/ui/issues/`: The Issues Strike Back [3/N]

Some `tests/ui/issues/` housekeeping, to trim down number of tests directly under `tests/ui/issues/`. Part of rust-lang#133895.

r? `@jieyouxu`
…jorn3

Make no_mangle on foreign items explicit instead of implicit

for a followup PR I'm working on I need some foreign items to mangle. I could add a new attribute: `no_no_mangle` or something silly like that but by explicitly putting `no_mangle` in the codegen fn attrs of foreign items we can default it to `no_mangle` and then easily remove it when we don't want it.

I guess you'd know about this r? `@bjorn3.` Shouldn't be too hard to review :)

Builds on rust-lang#144655 which should merge first.
… r=GuillaumeGomez

doc(library): Fix Markdown in `Iterator::by_ref`

This patch fixes the Markdown formatting in `std::core::iter::Iterator::by_ref`. Code is used inside a link without the backticks around the code.
Fix doc comment of File::try_lock and File::try_lock_shared

The doc comments of functions `File::try_lock` and `File::try_lock_shared` stabilized today in version 1.89.0 document an incorrect type of `Ok`.

The result type was changed in rust-lang#139343 after the latest change to the doc comments in rust-lang#136876.
move `type_check` out of `compute_regions`

A step towards rust-lang#139587. I don't think there's a clear reason for why MIR type check should be in `compute_regions` and this simplifies future PRs here.
Add a lot of NLL `known-bug` tests

r? `@jackh726`

As requested in rust-lang#143093 (review) this extracts most tests from that PR and expands upon them as described.

The handful of linked-list cursor-like tests will also turn into polonius=next known-bugs in rust-lang#143093 where the behavior w/r/t kills changes of course.
@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-clippy Relevant to the Clippy team. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Aug 7, 2025
@tgross35
Copy link
Contributor Author

tgross35 commented Aug 7, 2025

One iffy with a few always

@bors r+ rollup=never p=5

@bors
Copy link
Collaborator

bors commented Aug 7, 2025

📌 Commit 330a4e3 has been approved by tgross35

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 7, 2025
@bors
Copy link
Collaborator

bors commented Aug 7, 2025

⌛ Testing commit 330a4e3 with merge 017a6f7...

bors added a commit that referenced this pull request Aug 7, 2025
Rollup of 6 pull requests

Successful merges:

 - #144400 (`tests/ui/issues/`: The Issues Strike Back [3/N])
 - #144678 (Make no_mangle on foreign items explicit instead of implicit)
 - #145045 (doc(library): Fix Markdown in `Iterator::by_ref`)
 - #145046 (Fix doc comment of File::try_lock and File::try_lock_shared)
 - #145047 (move `type_check` out of `compute_regions`)
 - #145053 (Add a lot of NLL `known-bug` tests)

r? `@ghost`
`@rustbot` modify labels: rollup
@rust-log-analyzer
Copy link
Collaborator

The job test-various failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)
To only update this specific test, also pass `--test-args linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn.rs`

error: 1 errors occurred comparing output.
status: exit status: 1
command: env -u RUSTC_LOG_COLOR RUSTC_ICE="0" RUST_BACKTRACE="short" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/bin/rustc" "/checkout/tests/ui/linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn.rs" "-Zthreads=1" "-Zsimulate-remapped-rust-src-base=/rustc/FAKE_PREFIX" "-Ztranslate-remapped-path-to-local-path=no" "-Z" "ignore-directory-in-diagnostics-source-blocks=/cargo" "-Z" "ignore-directory-in-diagnostics-source-blocks=/checkout/vendor" "--sysroot" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2" "--target=wasm32-wasip1" "--check-cfg" "cfg(test,FALSE)" "--error-format" "json" "--json" "future-incompat" "-Ccodegen-units=1" "-Zui-testing" "-Zdeduplicate-diagnostics=no" "-Zwrite-long-types-to-disk=no" "-Cstrip=debuginfo" "--emit" "metadata" "-C" "prefer-dynamic" "--out-dir" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn" "-A" "unused" "-A" "internal_features" "-A" "unused_parens" "-A" "unused_braces" "-Crpath" "-Cdebuginfo=0" "-Lnative=/checkout/obj/build/wasm32-wasip1/native/rust-test-helpers"
stdout: none
--- stderr -------------------------------
error: attribute should be applied to a foreign function or static
##[error]  --> /checkout/tests/ui/linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn.rs:1:1
   |
---

error[E0455]: link kind `raw-dylib` is only supported on Windows targets
##[error]  --> /checkout/tests/ui/linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn.rs:13:34
   |
LL | #[link(name = "exporter", kind = "raw-dylib")]
   |                                  ^^^^^^^^^^^

error[E0391]: cycle detected when computing codegen attributes of `imported_function`
##[error]  --> /checkout/tests/ui/linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn.rs:16:5
   |
LL |     fn imported_function();
   |     ^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: ...which requires getting wasm import module map...
   = note: ...which requires looking up the native libraries of a linked crate...
   = note: ...which again requires computing codegen attributes of `imported_function`, completing the cycle
note: cycle used when checking that `` is well-formed
  --> /checkout/tests/ui/linkage-attr/raw-dylib/windows/link-ordinal-not-foreign-fn.rs:14:1
   |
LL | / extern "C" {
LL | |     #[link_ordinal(13)]
LL | |     fn imported_function();

@bors
Copy link
Collaborator

bors commented Aug 7, 2025

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 7, 2025
@tgross35 tgross35 closed this Aug 7, 2025
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 7, 2025
@tgross35 tgross35 deleted the rollup-1wm2fv2 branch August 7, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-attributes Area: Attributes (`#[…]`, `#![…]`) A-run-make Area: port run-make Makefiles to rmake.rs rollup A PR which is a rollup T-clippy Relevant to the Clippy team. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants