mirror of
https://github.com/fastfloat/fast_float.git
synced 2026-02-09 11:16:45 +08:00
Fixes for FASTFLOAT_IS_BIG_ENDIAN after PR #359 is added to original.
This commit is contained in:
parent
299c2741bc
commit
21d4ce1092
@ -544,7 +544,7 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
|
|
||||||
#ifndef FASTFLOAT_ONLY_POSITIVE_C_NUMBER_WO_INF_NAN
|
#ifndef FASTFLOAT_ONLY_POSITIVE_C_NUMBER_WO_INF_NAN
|
||||||
// Read sign
|
// Read sign
|
||||||
bool const negative = (*p == UC('-'));
|
auto const negative = (*p == UC('-'));
|
||||||
#ifdef FASTFLOAT_VISUAL_STUDIO
|
#ifdef FASTFLOAT_VISUAL_STUDIO
|
||||||
#pragma warning(push)
|
#pragma warning(push)
|
||||||
#pragma warning(disable : 4127)
|
#pragma warning(disable : 4127)
|
||||||
@ -557,7 +557,7 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
answer.ptr = first;
|
answer.ptr = first;
|
||||||
return answer;
|
return answer;
|
||||||
}
|
}
|
||||||
if ((*p == UC('-')) ||
|
if (negative ||
|
||||||
((chars_format_t(options.format & chars_format::allow_leading_plus)) &&
|
((chars_format_t(options.format & chars_format::allow_leading_plus)) &&
|
||||||
(*p == UC('+')))) {
|
(*p == UC('+')))) {
|
||||||
++p;
|
++p;
|
||||||
@ -571,7 +571,7 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
++p;
|
++p;
|
||||||
}
|
}
|
||||||
|
|
||||||
bool const has_leading_zeros = p > start_num;
|
auto const has_leading_zeros = p > start_num;
|
||||||
|
|
||||||
auto const *const start_digits = p;
|
auto const *const start_digits = p;
|
||||||
|
|
||||||
@ -592,26 +592,12 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
|
|
||||||
uint32_t digits;
|
uint32_t digits;
|
||||||
|
|
||||||
#if FASTFLOAT_HAS_IS_CONSTANT_EVALUATED && FASTFLOAT_HAS_BIT_CAST
|
if (cpp20_and_in_constexpr()) {
|
||||||
if (std::is_constant_evaluated()) {
|
|
||||||
uint8_t str[sizeof(uint32_t)];
|
uint8_t str[sizeof(uint32_t)];
|
||||||
for (uint_fast8_t j = 0; j != sizeof(uint32_t) && j != len; ++j) {
|
for (uint_fast8_t j = 0; j != sizeof(uint32_t) && j != len; ++j) {
|
||||||
str[j] = static_cast<uint8_t>(p[j]);
|
str[j] = static_cast<uint8_t>(p[j]);
|
||||||
}
|
}
|
||||||
digits = bit_cast<uint32_t>(str);
|
digits = bit_cast<uint32_t>(str);
|
||||||
#if FASTFLOAT_IS_BIG_ENDIAN
|
|
||||||
digits = byteswap(digits);
|
|
||||||
#endif
|
|
||||||
}
|
|
||||||
#else
|
|
||||||
if (false) {
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
else if (len >= 4) {
|
|
||||||
::memcpy(&digits, p, 4);
|
|
||||||
#if FASTFLOAT_IS_BIG_ENDIAN
|
|
||||||
digits = byteswap(digits);
|
|
||||||
#endif
|
|
||||||
} else {
|
} else {
|
||||||
uint32_t b0 = static_cast<uint8_t>(p[0]);
|
uint32_t b0 = static_cast<uint8_t>(p[0]);
|
||||||
uint32_t b1 = (len > 1) ? static_cast<uint8_t>(p[1]) : 0xFFu;
|
uint32_t b1 = (len > 1) ? static_cast<uint8_t>(p[1]) : 0xFFu;
|
||||||
@ -619,11 +605,14 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
uint32_t b3 = 0xFFu;
|
uint32_t b3 = 0xFFu;
|
||||||
digits = b0 | (b1 << 8) | (b2 << 16) | (b3 << 24);
|
digits = b0 | (b1 << 8) | (b2 << 16) | (b3 << 24);
|
||||||
}
|
}
|
||||||
|
#if FASTFLOAT_IS_BIG_ENDIAN
|
||||||
|
digits = byteswap(digits);
|
||||||
|
#endif
|
||||||
|
|
||||||
uint32_t magic =
|
uint32_t const magic =
|
||||||
((digits + 0x46464646u) | (digits - 0x30303030u)) & 0x80808080u;
|
((digits + 0x46464646u) | (digits - 0x30303030u)) & 0x80808080u;
|
||||||
uint32_t tz = (uint32_t)countr_zero_32(magic); // 7, 15, 23, 31, or 32
|
auto const tz = countr_zero_32(magic); // 7, 15, 23, 31, or 32
|
||||||
uint32_t nd = (tz == 32) ? 4 : (tz >> 3);
|
auto nd = am_digits(tz >> 3);
|
||||||
nd = std::min(nd, len);
|
nd = std::min(nd, len);
|
||||||
if (nd == 0) {
|
if (nd == 0) {
|
||||||
if (has_leading_zeros) {
|
if (has_leading_zeros) {
|
||||||
@ -631,14 +620,15 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
answer.ec = std::errc();
|
answer.ec = std::errc();
|
||||||
answer.ptr = p;
|
answer.ptr = p;
|
||||||
return answer;
|
return answer;
|
||||||
}
|
} else {
|
||||||
answer.ec = std::errc::invalid_argument;
|
answer.ec = std::errc::invalid_argument;
|
||||||
answer.ptr = first;
|
answer.ptr = first;
|
||||||
|
}
|
||||||
return answer;
|
return answer;
|
||||||
}
|
}
|
||||||
if (nd > 3) {
|
if (nd > 3) {
|
||||||
const UC *q = p + nd;
|
const UC *q = p + nd;
|
||||||
size_t rem = len - nd;
|
auto rem = len - nd;
|
||||||
while (rem) {
|
while (rem) {
|
||||||
if (*q < UC('0') || *q > UC('9'))
|
if (*q < UC('0') || *q > UC('9'))
|
||||||
break;
|
break;
|
||||||
@ -653,14 +643,15 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
digits ^= 0x30303030u;
|
digits ^= 0x30303030u;
|
||||||
digits <<= ((4 - nd) * 8);
|
digits <<= ((4 - nd) * 8);
|
||||||
|
|
||||||
uint32_t check = ((digits >> 24) & 0xff) | ((digits >> 8) & 0xff00) |
|
uint32_t const check = ((digits >> 24) & 0xff) |
|
||||||
|
((digits >> 8) & 0xff00) |
|
||||||
((digits << 8) & 0xff0000);
|
((digits << 8) & 0xff0000);
|
||||||
if (check > 0x00020505) {
|
if (check > 0x00020505) {
|
||||||
answer.ec = std::errc::result_out_of_range;
|
answer.ec = std::errc::result_out_of_range;
|
||||||
answer.ptr = p + nd;
|
answer.ptr = p + nd;
|
||||||
return answer;
|
return answer;
|
||||||
}
|
}
|
||||||
value = (uint8_t)((0x640a01 * digits) >> 24);
|
value = static_cast<uint8_t>((0x640a01 * digits) >> 24);
|
||||||
answer.ec = std::errc();
|
answer.ec = std::errc();
|
||||||
answer.ptr = p + nd;
|
answer.ptr = p + nd;
|
||||||
return answer;
|
return answer;
|
||||||
@ -682,7 +673,7 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
return answer;
|
return answer;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (len >= sizeof(uint32_t)) {
|
if (len >= std::numeric_limits<uint16_t>::digits10) {
|
||||||
auto digits = read_chars_to_unsigned<uint32_t>(p);
|
auto digits = read_chars_to_unsigned<uint32_t>(p);
|
||||||
if (is_made_of_four_digits_fast(digits)) {
|
if (is_made_of_four_digits_fast(digits)) {
|
||||||
auto v = parse_four_digits_unrolled(digits);
|
auto v = parse_four_digits_unrolled(digits);
|
||||||
@ -692,7 +683,7 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
answer.ec = std::errc::result_out_of_range;
|
answer.ec = std::errc::result_out_of_range;
|
||||||
const auto *q = p + 5;
|
const auto *q = p + 5;
|
||||||
while (q != pend && is_integer(*q)) {
|
while (q != pend && is_integer(*q)) {
|
||||||
q++;
|
++q;
|
||||||
}
|
}
|
||||||
answer.ptr = q;
|
answer.ptr = q;
|
||||||
return answer;
|
return answer;
|
||||||
@ -779,6 +770,7 @@ parse_int_string(UC const *p, UC const *pend, T &value,
|
|||||||
#ifdef FASTFLOAT_VISUAL_STUDIO
|
#ifdef FASTFLOAT_VISUAL_STUDIO
|
||||||
#pragma warning(push)
|
#pragma warning(push)
|
||||||
#pragma warning(disable : 4146)
|
#pragma warning(disable : 4146)
|
||||||
|
#pragma warning(disable : 4804)
|
||||||
#endif
|
#endif
|
||||||
// this weird workaround is required because:
|
// this weird workaround is required because:
|
||||||
// - converting unsigned to signed when its value is greater than signed max
|
// - converting unsigned to signed when its value is greater than signed max
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user