-
Notifications
You must be signed in to change notification settings - Fork 13k
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 #107980
Rollup of 7 pull requests #107980
Conversation
This symbol should be provided by Hermit.
…homcc reword descriptions of the deprecated int modules Based on recommendation by `@est31` here: rust-lang#107587 (comment) This is meant to make it more clear, when looking at the `std` or `core` docs, that these are deprecated modules - not deprecated integer types (a common misunderstanding). Before: ![image](https://user-images.githubusercontent.com/803701/216733011-fabc22e1-4e77-4a47-96e3-a765ac4690b6.png) After: ![image](https://user-images.githubusercontent.com/803701/216733660-02071ced-883d-4ab5-8c0a-d28547d1d5db.png)
…Simulacrum Add `array::map` benchmarks Since there were no previous benchmarks for `array::map`, and it is known to have mediocre/poor performance, add some simple benchmarks. These benchmarks vary the length of the array and size of each item.
…-Simulacrum Avoid copy-pasting the `ilog` panic string in a bunch of places I also ended up changing the implementations to `if let` because it doesn't work to ```rust self.checked_ilog2().unwrap_or_else(panic_for_nonpositive_argument) ``` due to the `!`. But as a bonus that meant I could remove the `rustc_allow_const_fn_unstable` too.
…ulacrum Add a doc note about why `Chain` is not `ExactSizeIterator` Inspired by <https://rust-lang.zulipchat.com/#narrow/stream/122651-general/topic/Why.20isn't.20Chain.3CA.2C.20B.3E.20an.20ExactSizeIterator.3F/near/327395874>.
…est, r=notriddle Update browser-ui-test version to 0.14.3 It brings a few fixes to the `NEAR` checks. The PR for it in [here](GuillaumeGomez/browser-UI-test#436). r? `@notriddle`
Hermit: Remove floor symbol This symbol should be provided by Hermit. It was introduced in 2019 (rust-lang#65167). Since 2020, Hermit provides these math functions on its own (hermit-os/hermit-rs#37). I think moving this to Hermit was merely an oversight. Related: * hermit-os/kernel#654 * hermit-os/hermit-rs#406 CC: `@stlankes`
…ngjubilee Fix unintentional UB in SIMD tests r? `@workingjubilee`
@bors r+ rollup=never p=5 |
☀️ Test successful - checks-actions |
📌 Perf builds for each rolled up PR: previous master: 96834f0231 In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
Finished benchmarking commit (2008188): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. |
Successful merges:
array::map
benchmarks #107915 (Addarray::map
benchmarks)ilog
panic string in a bunch of places #107961 (Avoid copy-pasting theilog
panic string in a bunch of places)Chain
is notExactSizeIterator
#107962 (Add a doc note about whyChain
is notExactSizeIterator
)Failed merges:
r? @ghost
@rustbot modify labels: rollup
Create a similar rollup