Skip to content

Conversation

@anonrig
Copy link
Member

@anonrig anonrig commented Sep 18, 2024

Before

BasicBench_AdaURL_url                201873 ns       201789 ns         3606 GHz=4.79146 cycle/byte=44.5044 cycles/url=1.07521k instructions/byte=111.317 instructions/cycle=2.50126 instructions/ns=11.9847 instructions/url=2.68938k ns/url=224.401 speed=102.008M/s time/byte=9.80319ns time/url=236.842ns url/s=4.22223M/s

After

BasicBench_AdaURL_url                195539 ns       195521 ns         3472 GHz=4.79119 cycle/byte=44.3816 cycles/url=1.07224k instructions/byte=111.317 instructions/cycle=2.50818 instructions/ns=12.0171 instructions/url=2.68937k ns/url=223.795 speed=105.278M/s time/byte=9.49867ns time/url=229.484ns url/s=4.3576M/s

@anonrig anonrig requested a review from lemire September 18, 2024 15:14
@anonrig anonrig force-pushed the yagiz/more-constexpr branch from f11d6f2 to cf327e3 Compare September 18, 2024 15:18
@anonrig anonrig requested a review from jasnell September 18, 2024 15:19
@anonrig anonrig force-pushed the yagiz/more-constexpr branch from cf327e3 to 916b8d4 Compare September 18, 2024 15:20
@lemire
Copy link
Member

lemire commented Sep 18, 2024

Let me see why it fails.

@lemire lemire mentioned this pull request Sep 18, 2024
@lemire
Copy link
Member

lemire commented Sep 18, 2024

See #738

@anonrig anonrig closed this in #738 Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants