-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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 9 pull requests #125820
Rollup of 9 pull requests #125820
Commits on May 28, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 077a821 - Browse repository at this point
Copy the full SHA 077a821View commit details -
Make drop-use fact collection simpler for
polonius
This shunts all the complexity of siphoning off the drop-use facts into `LivenessResults::add_extra_drop_facts()`, which may or may not be a good approach.
Configuration menu - View commit details
-
Copy full SHA for e155646 - Browse repository at this point
Copy the full SHA e155646View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8066ebc - Browse repository at this point
Copy the full SHA 8066ebcView commit details
Commits on May 30, 2024
-
Configuration menu - View commit details
-
Copy full SHA for dabd05b - Browse repository at this point
Copy the full SHA dabd05bView commit details -
Configuration menu - View commit details
-
Copy full SHA for fa563c1 - Browse repository at this point
Copy the full SHA fa563c1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 20174e6 - Browse repository at this point
Copy the full SHA 20174e6View commit details -
coverage: Instrument the RHS value of lazy logical operators
When a lazy logical operator (`&&` or `||`) occurs outside of an `if` condition, it normally doesn't have any associated control-flow branch, so we don't have an existing way to track whether it was true or false. This patch adds special code to handle this case, by inserting extra MIR blocks in a diamond shape after evaluating the RHS. This gives us a place to insert the appropriate marker statements, which can then be given their own counters.
Configuration menu - View commit details
-
Copy full SHA for 35a8746 - Browse repository at this point
Copy the full SHA 35a8746View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2ecb08a - Browse repository at this point
Copy the full SHA 2ecb08aView commit details
Commits on May 31, 2024
-
We do this for `&*` and `&mut*` already; might as well do it for raw pointers too.
Configuration menu - View commit details
-
Copy full SHA for 4b96e44 - Browse repository at this point
Copy the full SHA 4b96e44View commit details -
Also resolve the type of constants, even if we already turned it into…
… an error constant
Configuration menu - View commit details
-
Copy full SHA for 06c4cc4 - Browse repository at this point
Copy the full SHA 06c4cc4View commit details -
Configuration menu - View commit details
-
Copy full SHA for befcdec - Browse repository at this point
Copy the full SHA befcdecView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4abf26c - Browse repository at this point
Copy the full SHA 4abf26cView commit details -
Configuration menu - View commit details
-
Copy full SHA for c6022b4 - Browse repository at this point
Copy the full SHA c6022b4View commit details -
Add an alternate
--demangle
mode to coverage-dumpThe coverage-dump tool already needs `rustc_demangle` for its own purposes, so the amount of extra code needed for a demangle mode is very small.
Configuration menu - View commit details
-
Copy full SHA for 9abfebd - Browse repository at this point
Copy the full SHA 9abfebdView commit details -
Use
coverage-dump --demangle
as the demangler for coverage-run testsThis avoids the need to build `rust-demangler` when running coverage tests, since we typically need to build `coverage-dump` anyway.
Configuration menu - View commit details
-
Copy full SHA for 10ffc22 - Browse repository at this point
Copy the full SHA 10ffc22View commit details -
Use
Builder::tool_exe
to build the coverage-dump toolThis appears to be the canonical way to build a tool with the stage 0 compiler.
Configuration menu - View commit details
-
Copy full SHA for feb8f3c - Browse repository at this point
Copy the full SHA feb8f3cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 54b6849 - Browse repository at this point
Copy the full SHA 54b6849View commit details -
Configuration menu - View commit details
-
Copy full SHA for c0646f7 - Browse repository at this point
Copy the full SHA c0646f7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 57ca594 - Browse repository at this point
Copy the full SHA 57ca594View commit details -
Configuration menu - View commit details
-
Copy full SHA for 20699fe - Browse repository at this point
Copy the full SHA 20699feView commit details -
Rollup merge of rust-lang#125652 - amandasystems:you-dropped-somethin…
…g, r=oli-obk Revert propagation of drop-live information from Polonius rust-lang#64749 introduced a flow of drop-use data from Polonius to `LivenessResults::add_extra_drop_facts()`, which makes `LivenessResults` agree with Polonius on liveness in the presence of free regions that may be dropped. Later changes accidentally removed this flow. This PR restores it.
Configuration menu - View commit details
-
Copy full SHA for e895233 - Browse repository at this point
Copy the full SHA e895233View commit details -
Rollup merge of rust-lang#125730 - mu001999-contrib:clippy-fix, r=oli…
…-obk Apply `x clippy --fix` and `x fmt` on Rustc <!-- If this PR is related to an unstable feature or an otherwise tracked effort, please link to the relevant tracking issue here. If you don't know of a related tracking issue or there are none, feel free to ignore this. This PR will get automatically assigned to a reviewer. In case you would like a specific user to review your work, you can assign it to them by using r? <reviewer name> --> Just run `x clippy --fix` and `x fmt`, and remove some changes like `impl Default`.
Configuration menu - View commit details
-
Copy full SHA for fe6ecc2 - Browse repository at this point
Copy the full SHA fe6ecc2View commit details -
Rollup merge of rust-lang#125756 - Zalathar:branch-on-bool, r=oli-obk
coverage: Optionally instrument the RHS of lazy logical operators (This is an updated version of rust-lang#124644 and rust-lang#124402. Fixes rust-lang#124120.) When `||` or `&&` is used outside of a branching context (such as the condition of an `if`), the rightmost value does not directly influence any branching decision, so branch coverage instrumentation does not treat it as its own true-or-false branch. That is a correct and useful interpretation of “branch coverage”, but might be undesirable in some contexts, as described at rust-lang#124120. This PR therefore adds a new coverage level `-Zcoverage-options=condition` that behaves like branch coverage, but also adds additional branch instrumentation to the right-hand-side of lazy boolean operators. --- As discussed at rust-lang#124120 (comment), this is mainly intended as an intermediate step towards fully-featured MC/DC instrumentation. It's likely that we'll eventually want to remove this coverage level (rather than stabilize it), either because it has been incorporated into MC/DC instrumentation, or because it's getting in the way of future MC/DC work. The main appeal of landing it now is so that work on tracking conditions can proceed concurrently with other MC/DC-related work. ```@rustbot``` label +A-code-coverage
Configuration menu - View commit details
-
Copy full SHA for 45147ef - Browse repository at this point
Copy the full SHA 45147efView commit details -
Rollup merge of rust-lang#125773 - GuillaumeGomez:migrate-run-make-cd…
…ylib, r=jieyouxu Migrate run make cdylib Part of rust-lang#121876. r? `@jieyouxu`
Configuration menu - View commit details
-
Copy full SHA for e3042f7 - Browse repository at this point
Copy the full SHA e3042f7View commit details -
Rollup merge of rust-lang#125776 - compiler-errors:translate-args, r=…
…lcnr Stop using `translate_args` in the new solver It was unnecessary and also sketchy, since it was doing an out-of-search-graph fulfillment loop. Added a test for the only really minor subtlety of translating args, though not sure if it was being tested before, though I wouldn't be surprised if it wasn't. r? lcnr
Configuration menu - View commit details
-
Copy full SHA for 79955a1 - Browse repository at this point
Copy the full SHA 79955a1View commit details -
Rollup merge of rust-lang#125796 - scottmcm:more-inst-simplify, r=oli…
…-obk Also InstSimplify `&raw*` We do this for `&*` and `&mut*` already; might as well do it for raw pointers too. r? mir-opt
Configuration menu - View commit details
-
Copy full SHA for 12377db - Browse repository at this point
Copy the full SHA 12377dbView commit details -
Rollup merge of rust-lang#125807 - oli-obk:resolve_const_types, r=com…
…piler-errors Also resolve the type of constants, even if we already turned it into an error constant error constants can still have arbitrary types, and in this case it was turned into an error constant because there was an infer var in the *type* not the *const*. fixes rust-lang#125760
Configuration menu - View commit details
-
Copy full SHA for cc5c36f - Browse repository at this point
Copy the full SHA cc5c36fView commit details -
Rollup merge of rust-lang#125808 - GuillaumeGomez:migrate-run-make-c-…
…link-to-rust-dylib, r=jieyouxu Migrate `run-make/c-link-to-rust-dylib` to `rmake.rs` Part of rust-lang#121876. First commit comes from rust-lang#125773. r? `@jieyouxu`
Configuration menu - View commit details
-
Copy full SHA for 6b6d5f2 - Browse repository at this point
Copy the full SHA 6b6d5f2View commit details -
Rollup merge of rust-lang#125816 - Zalathar:demangler, r=oli-obk
Don't build the `rust-demangler` binary for coverage tests The coverage-run tests invoke `llvm-cov`, which requires us to specify a command-line demangler that it can use to demangle Rust symbol names. Historically this used `src/tools/rust-demangler`, which means that we currently build two different command-line tools to help with the coverage tests (`rust-demangler` and `coverage-dump`). However, it occurred to me that if we add a demangler mode to `coverage-dump` (which is only a handful of lines and no extra dependencies), then we only need to build one helper binary for the coverage tests, and there is no need for tests to build `rust-demangler` at all. --- Note that the `rust-demangler` binary is separate from the `rustc-demangle` crate (which both `rust-demangler` and `coverage-dump` use as a dependency to do the actual demangling). --- So the main benefits/motivations here are: - Slightly faster builds after a fresh checkout or bootstrap bump. - Making it clear that currently no tests actually need the `rust-demangler` binary, since the coverage tests can use their own tool instead.
Configuration menu - View commit details
-
Copy full SHA for 85a859d - Browse repository at this point
Copy the full SHA 85a859dView commit details