-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Rollup of 11 pull requests #149798
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
Rollup of 11 pull requests #149798
Conversation
Instead of allowing them to be missing and using some placeholder "(none)" value instead.
Instead of possibly falling back to "(none)" when they are not specified.
We either have the value of a flag specified, or we don't. Use `Option<...>` to represent that -- don't invent a new "(none)" sentinel value...
This derive was an artifact of test-only method `Cache::all` wanting to automatically sort its output to hide HashMap iteration order. We can achieve an equivalent result by requiring the caller to provide a projection function that returns results that _are_ sortable.
Replaced _binder with _
Even if a crate is marked as #![no_builtins], we can still generate bitcode for rlib, but we cannot emit bitcode to the linker in rustc's LTO.
…ess-private-fields, r=Kivooeo Suppress the error for private fields with non_exhaustive attribute Fixes rust-lang#147513
…szelmann Emit `check-cfg` lints during attribute parsing rather than evaluation The goal of this PR is to make the `eval_config_entry` not have any side effects, by moving the check-cfg lints to the attribute parsing. This also helps ensure we do emit the lint in situations where the attribute happens to be parsed, but never evaluated. cc ``@jdonszelmann`` ``@Urgau`` for a vibe check if you feel like it
Add release notes for 1.92.0 cc ``@rust-lang/release`` r? ``@BoxyUwU``
rustdoc book: mention inner doc attribute Couldn't find this documented anywhere else.
lint: emit proper diagnostic for unsafe binders in improper_ctypes instead of ICE Fixes rust-lang#149719 This PR replaces the `todo!("FIXME(unsafe_binder)")` branch in the `improper_ctypes` lint with a proper diagnostic. Previously, using an unsafe binder inside an extern `"C"` block caused an internal compiler error. This fix now ensures that the compiler now emits a clear and stable error message explaining that types containing unsafe binders are not yet supported in FFI. A new UI test `(unsafe-binder-basic.rs)` is included to ensure this behavior remains stable and prevent regressions.
Retire `opt_str2` from compiletest cli parsing We have `Option<..>`, we don't need to invent "(none)" as option-at-home. - More specifically, when some test suite expect certain values to be present, they should `.expect(..)` the value, and not potentially receive some "(none)" in some cases. r? Zalathar
bootstrap: Use a `CompiletestMode` enum instead of bare strings This gives better static checking when dealing with compiletest modes in bootstrap, especially for those modes that overlap with the name of a test suite directory. r? jieyouxu (or bootstrap)
…petrochenkov Add inline attribute to generated delegation function if needed This PR adds a functionality to add #[inline(always)] attribute to generated functions from delegation, if some #[inline(..)] attribute is already specified on the delegation then we do nothing. This PR is a part of the delegation feature rust-lang#118212 and addresses `Add implicit #[inline] unless specified otherwise` item. r? ``@petrochenkov``
…to, r=jieyouxu test: Add a test for 146133 This is a test without using `-Zbuild-std` for rust-lang#146133. There is another test using `-Zbuild-std`: rust-lang/cargo#16277. Even if a crate is marked as `#![no_builtins]`, we can still generate bitcode for rlib, but we cannot emit bitcode to the linker in rustc's LTO.
Fix typo "an" → "and"
…, r=Kobzol Remove `[no-mentions]` handler in the triagebot config This PR removes the `[no-mentions]` handler in the triagebot config as [GitHub is removing notifications for ``@-mentions`` in commit messages on December 8th 2025.](https://github.blog/changelog/2025-11-07-removing-notifications-for-mentions-in-commit-messages/) cf. rust-lang/triagebot#2225
|
@bors r+ rollup=never p=5 |
|
☀️ Test successful - checks-actions |
|
📌 Perf builds for each rolled up PR:
previous master: 018d26972e In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 018d269 (parent) -> d5525a7 (this PR) Test differencesShow 17 test diffsStage 1
Stage 2
Additionally, 6 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard d5525a73009e2c61b09daa69df79064530bd4dcf --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (d5525a7): comparison URL. Overall result: ❌✅ regressions and improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary 0.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -2.3%, secondary 3.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.0%, secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 472.223s -> 472.272s (0.01%) |
Successful merges:
check-cfglints during attribute parsing rather than evaluation #149215 (Emitcheck-cfglints during attribute parsing rather than evaluation)opt_str2from compiletest cli parsing #149754 (Retireopt_str2from compiletest cli parsing)CompiletestModeenum instead of bare strings #149755 (bootstrap: Use aCompiletestModeenum instead of bare strings)[no-mentions]handler in the triagebot config #149782 (Remove[no-mentions]handler in the triagebot config)Failed merges:
async fnwith adyn Traitreturn type #148491 ( Correctly provide suggestions when encounteringasync fnwith adyn Traitreturn type)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup