diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2023-08-01 17:39:12 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-08-01 17:39:12 +0200 |
| commit | 00ad3ccae694a5da21e7dee9478d87ea4f7958c5 (patch) | |
| tree | 85ef245b66ed1f9e881b7f4e750c3a3af82245fc /compiler/rustc_codegen_llvm/src/context.rs | |
| parent | fa33d6e1853b90e39dab2b0193fc02d18286cafc (diff) | |
| parent | b84942a0fe11c06a94cfe46ee787084e80c51fb9 (diff) | |
| download | rust-00ad3ccae694a5da21e7dee9478d87ea4f7958c5.tar.gz rust-00ad3ccae694a5da21e7dee9478d87ea4f7958c5.zip | |
Rollup merge of #114306 - ttsugriy:push_str, r=wesleywiser
[rustc_data_structures][perf] Simplify base_n::push_str.
This minor change removes the need to reverse resulting digits. Since reverse is O(|digit_num|) but bounded by 128, it's unlikely to be a noticeable in practice. At the same time, this code is also a 1 line shorter, so combined with tiny perf win, why not?
I ran https://gist.github.com/ttsugriy/ed14860ef597ab315d4129d5f8adb191 on M1 macbook air and got a small improvement
```
Running benches/base_n_benchmark.rs (target/release/deps/base_n_benchmark-825fe5895b5c2693)
push_str/old time: [14.180 µs 14.313 µs 14.462 µs]
Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
4 (4.00%) high mild
1 (1.00%) high severe
push_str/new time: [13.741 µs 13.839 µs 13.973 µs]
Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
3 (3.00%) high mild
5 (5.00%) high severe
```
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/context.rs')
0 files changed, 0 insertions, 0 deletions