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 8 pull requests #122803
Rollup of 8 pull requests #122803
Commits on Mar 19, 2024
-
Configuration menu - View commit details
-
Copy full SHA for a2c74b8 - Browse repository at this point
Copy the full SHA a2c74b8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5e4cc6f - Browse repository at this point
Copy the full SHA 5e4cc6fView commit details -
SeqCst->Relaxed for proc_macro bridge counter.
Relaxed is enough here.
Configuration menu - View commit details
-
Copy full SHA for bf3debe - Browse repository at this point
Copy the full SHA bf3debeView commit details -
SeqCst->{Release,Acquire} for alloc error hook.
SeqCst is unnecessary.
Configuration menu - View commit details
-
Copy full SHA for 904fef0 - Browse repository at this point
Copy the full SHA 904fef0View commit details -
SeqCst->Relaxed for FIRST_PANIC.
Relaxed is enough to make sure this `swap` results in `true` only once.
Configuration menu - View commit details
-
Copy full SHA for 9f25a04 - Browse repository at this point
Copy the full SHA 9f25a04View commit details -
SeqCst->{Release,Acquire} in xous mutex.
No need for SeqCst. Release+Acquire is the right memory ordering for a mutex.
Configuration menu - View commit details
-
Copy full SHA for eb96698 - Browse repository at this point
Copy the full SHA eb96698View commit details -
Use less restricted memory ordering in thread_parking::pthread.
SeqCst is unnecessary here.
Configuration menu - View commit details
-
Copy full SHA for 516684c - Browse repository at this point
Copy the full SHA 516684cView commit details -
SeqCst->{Release,Acquire} in sys_common::thread_local_key.
SeqCst is unnecessary here.
Configuration menu - View commit details
-
Copy full SHA for e43aef0 - Browse repository at this point
Copy the full SHA e43aef0View commit details -
SeqCst->{Release,Acquire} for wasm DropLock.
SeqCst is unnecessary. Release+Acquire is the right ordering for a mutex.
Configuration menu - View commit details
-
Copy full SHA for 46bb073 - Browse repository at this point
Copy the full SHA 46bb073View commit details -
SeqCst->Relaxed in pal::windows::pipe.
Relaxed is enough to ensure fetch_add(1) returns each integer exactly once.
Configuration menu - View commit details
-
Copy full SHA for 60ad490 - Browse repository at this point
Copy the full SHA 60ad490View commit details -
SeqCst->{Release,Acquire} for xous DropLock.
SeqCst is unnecessary. Release+Acquire is the right ordering for a mutex.
Configuration menu - View commit details
-
Copy full SHA for 69a4d77 - Browse repository at this point
Copy the full SHA 69a4d77View commit details -
SeqCst->Relaxed for xous set_nonblocking.
The SeqCst wasn't synchronizing with anything. Relaxed is enough.
Configuration menu - View commit details
-
Copy full SHA for 5a594f7 - Browse repository at this point
Copy the full SHA 5a594f7View commit details -
rustc is just one tool/executable, even if at the center of the toolchain
Configuration menu - View commit details
-
Copy full SHA for 75a5196 - Browse repository at this point
Copy the full SHA 75a5196View commit details
Commits on Mar 20, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 70206f0 - Browse repository at this point
Copy the full SHA 70206f0View commit details -
coverage: Remove incorrect assertions from counter allocation
These assertions detect situations where a BCB node would have both a physical counter and one or more in-edge counters/expressions. For most BCBs that situation would indicate an implementation bug. However, it's perfectly fine in the case of a BCB having an edge that loops back to itself. Given the complexity and risk involved in fixing the assertions, and the fact that nothing relies on them actually being true, this patch just removes them instead.
Configuration menu - View commit details
-
Copy full SHA for 85bec7a - Browse repository at this point
Copy the full SHA 85bec7aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2f21e4f - Browse repository at this point
Copy the full SHA 2f21e4fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 92f668c - Browse repository at this point
Copy the full SHA 92f668cView commit details -
Use less restricted memory ordering in xous::thread_local_key.
SeqCst isn't necessary in any of these cases.
Configuration menu - View commit details
-
Copy full SHA for 8b519f9 - Browse repository at this point
Copy the full SHA 8b519f9View commit details -
SeqCst->Relaxed in std::net::test.
Relaxed is enough to have fetch_add(1) return each value only once (until it wraps around).
Configuration menu - View commit details
-
Copy full SHA for b45a725 - Browse repository at this point
Copy the full SHA b45a725View commit details -
SeqCst->Relaxed in thread local test.
Relaxed memory ordering is fine because spawn()/join() already provides all the synchronization we need.
Configuration menu - View commit details
-
Copy full SHA for acddc55 - Browse repository at this point
Copy the full SHA acddc55View commit details -
SeqCst->Relaxed in condvar test.
Relaxed is enough here. Synchronization is done by the mutex.
Configuration menu - View commit details
-
Copy full SHA for 3462175 - Browse repository at this point
Copy the full SHA 3462175View commit details -
Configuration menu - View commit details
-
Copy full SHA for 98e6655 - Browse repository at this point
Copy the full SHA 98e6655View commit details -
Configuration menu - View commit details
-
Copy full SHA for a845246 - Browse repository at this point
Copy the full SHA a845246View commit details -
Configuration menu - View commit details
-
Copy full SHA for 19e0ea4 - Browse repository at this point
Copy the full SHA 19e0ea4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0dc006b - Browse repository at this point
Copy the full SHA 0dc006bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 352587a - Browse repository at this point
Copy the full SHA 352587aView commit details -
compiletest: Introduce remove_and_create_dir_all() helper
The code let _ = fs::remove_dir_all(&dir); create_dir_all(&dir).unwrap(); is duplicated in 7 places. Let's introduce a helper.
Configuration menu - View commit details
-
Copy full SHA for c3cc6c1 - Browse repository at this point
Copy the full SHA c3cc6c1View commit details
Commits on Mar 21, 2024
-
Rollup merge of rust-lang#122545 - Alexendoo:unused-qualifications, r…
…=petrochenkov Ignore paths from expansion in `unused_qualifications` If any of the path segments are from an expansion the lint is skipped currently, but a path from an expansion where all of the segments are passed in would not be. Doesn't seem that likely to occur but it could happen
Configuration menu - View commit details
-
Copy full SHA for 31adfd7 - Browse repository at this point
Copy the full SHA 31adfd7View commit details -
Rollup merge of rust-lang#122729 - m-ou-se:relax, r=Amanieu
Relax SeqCst ordering in standard library. Every single SeqCst in the standard library is unnecessary. In all cases, Relaxed or Release+Acquire was sufficient. As I [wrote](https://marabos.nl/atomics/memory-ordering.html#common-misconceptions) in my book on atomics: > [..] when reading code, SeqCst basically tells the reader: "this operation depends on the total order of every single SeqCst operation in the program," which is an incredibly far-reaching claim. The same code would likely be easier to review and verify if it used weaker memory ordering instead, if possible. For example, Release effectively tells the reader: "this relates to an acquire operation on the same variable," which involves far fewer considerations when forming an understanding of the code. > > It is advisable to see SeqCst as a warning sign. Seeing it in the wild often means that either something complicated is going on, or simply that the author did not take the time to analyze their memory ordering related assumptions, both of which are reasons for extra scrutiny. r? ````@Amanieu```` ````@joboet````
Configuration menu - View commit details
-
Copy full SHA for 43ad753 - Browse repository at this point
Copy the full SHA 43ad753View commit details -
Rollup merge of rust-lang#122740 - tshepang:patch-1, r=clubby789
use more accurate terminology rustc is just one tool/executable, even if at the center of the toolchain
Configuration menu - View commit details
-
Copy full SHA for d53df5a - Browse repository at this point
Copy the full SHA d53df5aView commit details -
Rollup merge of rust-lang#122749 - aliemjay:region-err, r=compiler-er…
…rors make `type_flags(ReError) & HAS_ERROR` Self-explanatory. `TypeVisitableExt::references_error(ReError)` incorrectly returned `false`.
Configuration menu - View commit details
-
Copy full SHA for 4e792df - Browse repository at this point
Copy the full SHA 4e792dfView commit details -
Rollup merge of rust-lang#122764 - Zalathar:loopy, r=oli-obk
coverage: Remove incorrect assertions from counter allocation These assertions detect situations where a BCB node (in the coverage graph) would have both a physical counter and one or more in-edge counters/expressions. For most BCBs that situation would indicate an implementation bug. However, it's perfectly fine in the case of a BCB having an edge that loops back to itself. Given the complexity and risk involved in fixing the assertions, and the fact that nothing relies on them actually being true, this patch just removes them instead. Fixes rust-lang#122738. `````@rustbot````` label +A-code-coverage
Configuration menu - View commit details
-
Copy full SHA for c6a4922 - Browse repository at this point
Copy the full SHA c6a4922View commit details -
Rollup merge of rust-lang#122765 - workingjubilee:test-for-vec-handli…
…ng-usize-max, r=Nilstrieb Add `usize::MAX` arg tests for Vec Tests to prevent recurrence of the UB from the rust-lang#122760 issue. I skipped the `with_capacity`, `drain`, `reserve`, etc. APIs because they actually had a good assortment of tests earlier in the same file. r? Nilstrieb
Configuration menu - View commit details
-
Copy full SHA for 7a5ffcc - Browse repository at this point
Copy the full SHA 7a5ffccView commit details -
Rollup merge of rust-lang#122776 - GuillaumeGomez:rename-hir-let, r=o…
…li-obk Rename `hir::Let` into `hir::LetExpr` As discussed on [zulip](https://rust-lang.zulipchat.com/#narrow/stream/131828-t-compiler/topic/Improve.20naming.20of.20.60ExprKind.3A.3ALet.60.3F). r? `````@Zalathar`````
Configuration menu - View commit details
-
Copy full SHA for afdbad8 - Browse repository at this point
Copy the full SHA afdbad8View commit details -
Rollup merge of rust-lang#122786 - Enselic:remove_and_create_dir_all,…
… r=onur-ozkan compiletest: Introduce `remove_and_create_dir_all()` helper The code let _ = fs::remove_dir_all(&dir); create_dir_all(&dir).unwrap(); is duplicated in 7 places. Let's introduce a helper.
Configuration menu - View commit details
-
Copy full SHA for f25397a - Browse repository at this point
Copy the full SHA f25397aView commit details