-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
Rollup of 7 pull requests #151325
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 7 pull requests #151325
Conversation
Allow invoking all help options at once Implements rust-lang#150442 Help messages are printed in the order they are invoked, but only once (e.g. `--help -C help --help` prints regular help then codegen help). Lint groups (`-Whelp`) are always printed last due to necessarily coming later in the compiler pipeline. Adds `help` flags to `-C` and `-Z` to avoid an error in `build_session_options`. cc @bjorn3 [#t-compiler/major changes > Allow combining &rust-lang#96;--help -C help -Z help -… compiler-team#954 @ 💬](https://rust-lang.zulipchat.com/#narrow/channel/233931-t-compiler.2Fmajor-changes/topic/Allow.20combining.20.60--help.20-C.20help.20-Z.20help.20-.E2.80.A6.20compiler-team.23954/near/564358190) - this currently maintains the behaviour of unrecognized options always failing-fast, as this happens within `getopt`s parsing, before we can even check if help options are present.
Added mGCA related tests Add regression tests for subsequent mGCA tasks edit: resolve: rust-lang#147415 `tests\ui\const-generics\mgca\size-of-generic-ptr-in-array-len.rs` resolve: rust-lang#141014 `tests\ui\const-generics\mgca\assoc-const-projection-in-bound.rs` and crashes: 138226, 138226-2, 149809, 150960
Explicitly list crate level attrs r? @jdonszelmann
…, r=mati865 Fix ICE on inconsistent import resolution with macro-attributed extern crate Fixes rust-lang#151213 using issue_145575_hack_applied in the same way as in rust-lang#149860.
Normalize type_const items even with feature `generic_const_exprs` Fixes rust-lang#151251 With feature `generic_const_exprs` enabled, consts with `#[type_const]` won't be normalized even if they need. Then ICE happens when CTFE tries to evaluate such const without body. Fix this by normalizing such consts even with feature `generic_const_exprs` enabled. r? @BoxyUwU
Use `find_attr` instead of `attr::contains_name` in `lower_const_item_rhs` Fixes rust-lang#151250 `attr::contains_name` uses `AttributeExt::name()` to filter, but for `hir::Attribute::Parsed`, this method will return `None`, and then `attr::contains_name` will return `false` here. So that the previous logic cannot work as expected. r? @BoxyUwU
Port #![no_main] to the attribute parser. Tracking issue: rust-lang#131229 r? @JonathanBrouwer
|
@bors r+ rollup=never p=5 |
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 2b112efccc 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 2b112ef (parent) -> 0a3cd3b (this PR) Test differencesShow 30 test diffsStage 1
Stage 2
Additionally, 4 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 0a3cd3b6b6e1fa8fd3c75c1d13d2e22e64273f49 --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 (0a3cd3b): comparison URL. Overall result: ❌ regressions - 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 -1.0%, secondary -3.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 474.393s -> 474.37s (-0.00%) |
Successful merges:
generic_const_exprs#151275 (Normalize type_const items even with featuregeneric_const_exprs)find_attrinstead ofattr::contains_nameinlower_const_item_rhs#151288 (Usefind_attrinstead ofattr::contains_nameinlower_const_item_rhs)r? @ghost
Create a similar rollup