Skip to content

c-variadic: remove stale cfg(bootstrap) comment - #165496

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
jieyouxu:jieyouxu/remove-stale-cfg-bootstrap
Nov 5, 2026
Merged

rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
jieyouxu:jieyouxu/remove-stale-cfg-bootstrap

Conversation

@jieyouxu

@jieyouxu jieyouxu commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

This can't unwrap, e.g. under no_core you're not guaranteed to have that lang item.

Noticed when grepping for cfg(bootstrap) when reviewing an unrelated PR.

r? @folkertdev (or compiler)

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 5, 2026
This can't unwrap, e.g. `no_core` you're not guaranteed to have that
lang item.
@jieyouxu
jieyouxu force-pushed the jieyouxu/remove-stale-cfg-bootstrap branch from aea2a34 to b17c1a8 Compare November 5, 2026 12:27
@folkertdev

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors

rust-bors Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit b17c1a8 has been tentatively approved by folkertdev

It will be put into the queue for this repository once PR CI succeeds.

@rust-bors rust-bors Bot 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 Nov 5, 2026
@rust-bors

rust-bors Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

⌛ Testing commit b17c1a8 with merge 2af19de...

Workflow: https://github.com/rust-lang/rust/actions/runs/42040750754

rust-bors Bot pushed a commit that referenced this pull request Nov 5, 2026
… r=folkertdev

c-variadic: remove stale `cfg(bootstrap)` comment

This can't unwrap, e.g. under `no_core` you're not guaranteed to have that lang item.

Noticed when grepping for `cfg(bootstrap)` when reviewing an unrelated PR.

r? @folkertdev (or compiler)
@JonathanBrouwer

Copy link
Copy Markdown
Member

@bors yield
Yielding to enclosing rollup

@rust-bors

rust-bors Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Auto build was cancelled. Cancelled workflows:

The next pull request likely to be tested is #165510.

rust-bors Bot pushed a commit that referenced this pull request Nov 5, 2026
…uwer

Rollup of 8 pull requests

Successful merges:

 - #165496 (c-variadic: remove stale `cfg(bootstrap)` comment)
 - #164716 (Lower UEFI targets through LLVM's *uefi triple)
 - #165497 (fix unsoundness when compiling `[diverging_const; _]`)
 - #165275 (Build TPDE with local Clang when available)
 - #165467 (Use `emit_ptr_va_arg` for `va_arg` on MSP430)
 - #165483 (compiletest: Fix the edition of minicore to the latest stable one)
 - #165498 (LoongArch: Fix direct-access-external-data test)
 - #165501 (rustdoc: Don't add a border on an empty `:target` element)
rust-bors Bot pushed a commit that referenced this pull request Nov 5, 2026
Rollup merge of #165496 - jieyouxu:jieyouxu/remove-stale-cfg-bootstrap, r=folkertdev

c-variadic: remove stale `cfg(bootstrap)` comment

This can't unwrap, e.g. under `no_core` you're not guaranteed to have that lang item.

Noticed when grepping for `cfg(bootstrap)` when reviewing an unrelated PR.

r? @folkertdev (or compiler)
@rust-bors
rust-bors Bot merged commit 030deb0 into rust-lang:main Nov 5, 2026
14 of 15 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Nov 5, 2026
@jieyouxu
jieyouxu deleted the jieyouxu/remove-stale-cfg-bootstrap branch November 6, 2026 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants