-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
std.mem
: Don't use vectorized path for freestanding targets in indexOfSentinel()
#23182
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
base: master
Are you sure you want to change the base?
Conversation
std
: Remove invalid page size assumptionsstd.mem
: Don't use vectorized path for freestanding targets in indexOfSentinel()`
std.mem
: Don't use vectorized path for freestanding targets in indexOfSentinel()`std.mem
: Don't use vectorized path for freestanding targets in indexOfSentinel()
What makes the page size assumptions invalid? Also this is not a bug fix, this is breaking. |
The key points from the previous discussion are:
|
4f43918
to
fc88592
Compare
fc88592
to
ba2c859
Compare
ba2c859
to
dfbdef9
Compare
This removes the invalid page size assumptions made in
std.mem.indexOfSentinel()
for freestanding targets. This is what originally prompted the commits that I'm reverting, which were added in #22488 and #23043. See the discussion: #22488 (review)See also #23184.