BurntSushi/memchr
Ver no GitHubConsider hinting to the compiler that indices returned from `memchr()` are within bounds of the haystack
Open
#195 aberto em 1 de dez. de 2025
enhancementhelp wanted
Métricas do repositório
- Stars
- (1.457 stars)
- Métricas de merge de PR
- (Mesclagem média 18m) (2 fundiu PRs em 30d)
Description
Currently, indexing into the haystack with the index returned by memchr usually generates a bounds check, because the compiler doesn't know that it's valid. However, if an assert_unchecked(i < haystack.len()) is inserted, the compiler has enough information to elide the check (godbolt).
Pros:
- Reduces bounds-checking in downstream code; this doesn't remove a ton of overhead but could possibly allow for further optimizations that had been inhibited by the branch.
Cons:
assert_unchecked()is only available since Rust 1.81, so would either have to feature-gate this, do rust version detection, or (most likely) useif !cond { unreachable_unchecked(); }to polyfill it. The last option should be equivalent toassert_unchecked(), but I'm not sure if it's treated exactly the same by the compiler.- Returning an out-of-bound index becomes UB rather than just a logic error. I assume there was already downstream code relying on this, though, and also it'd probably already be UB internally.