]> git.lizzy.rs Git - rust.git/commit - src/tools/miri
Auto merge of #97526 - Nilstrieb:unicode-is-printable-fastpath, r=joshtriplett
authorbors <bors@rust-lang.org>
Tue, 31 May 2022 09:34:00 +0000 (09:34 +0000)
committerbors <bors@rust-lang.org>
Tue, 31 May 2022 09:34:00 +0000 (09:34 +0000)
commitdcbd5f5134e396353207f1f79e50794b83cee7d2
treead8a495092c69694241f4b3de984d52ad0eb79bf
parentd35d972e6974d40d30362344ea619a5b560aae20
parent3358a41acbcb82daf3cd71197b2a053152a9a376
Auto merge of #97526 - Nilstrieb:unicode-is-printable-fastpath, r=joshtriplett

Add unicode fast path to `is_printable`

Before, it would enter the full expensive check even for normal ascii characters. Now, it skips the check for the ascii characters in `32..127`. This range was checked manually from the current behavior.

I ran the `tracing` test suite in miri, and it was really slow. I looked at a profile, and miri spent most of the time in `core::char::methods::escape_debug_ext`, where half of that was dominated by `core::unicode::printable::is_printable`. So I optimized it here.

The tracing profile:
![The tracing profile](https://user-images.githubusercontent.com/48135649/170883650-23876e7b-3fd1-4e8b-9001-47672e06d914.svg)