Skip to content

Commit 7726265

Browse files
authored
Rollup merge of rust-lang#95831 - redzic:xor-uppercase, r=workingjubilee
Use bitwise XOR in to_ascii_uppercase This saves an instruction compared to the previous approach, which was to unset the fifth bit with bitwise OR. Comparison of generated assembly on x86: https://godbolt.org/z/GdfvdGs39 This can also affect autovectorization, saving SIMD instructions as well: https://godbolt.org/z/cnPcz75T9 Not sure if `u8::to_ascii_lowercase` should also be changed, since using bitwise OR for that function does not require an extra bitwise negate since the code is setting a bit rather than unsetting a bit. `char::to_ascii_uppercase` already uses XOR, so no change seems to be required there.
2 parents 2464ea2 + 1e6365d commit 7726265

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

library/core/src/num/mod.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -299,8 +299,8 @@ impl u8 {
299299
#[rustc_const_stable(feature = "const_ascii_methods_on_intrinsics", since = "1.52.0")]
300300
#[inline]
301301
pub const fn to_ascii_uppercase(&self) -> u8 {
302-
// Unset the fifth bit if this is a lowercase letter
303-
*self & !((self.is_ascii_lowercase() as u8) * ASCII_CASE_MASK)
302+
// Toggle the fifth bit if this is a lowercase letter
303+
*self ^ ((self.is_ascii_lowercase() as u8) * ASCII_CASE_MASK)
304304
}
305305

306306
/// Makes a copy of the value in its ASCII lower case equivalent.

0 commit comments

Comments
 (0)