Changes imported from Abseil "staging" branch:

- c77facb7ba7c12d656e213e5d63d7dcd9d1f56ba Present the Writer* methods as an option, rather than an ... by Abseil Team <absl-team@google.com>
  - c6004d718f185868dee2ec48b4de6425399b062d Split exception_safety_testing out into its own cmake rul... by Jon Cohen <cohenjon@google.com>
  - 9aa6fc7b34674472d808ed8021f3b20faea82f78 Downstream https://github.com/abseil/abseil-cpp/pull/69 by Jon Cohen <cohenjon@google.com>
  - 13d340269f284b815cc409d2271099e535f4cba4 Internal change by Jon Cohen <cohenjon@google.com>
  - 51840a9243ac9d47eb3c177398d10bb3341f3230 Fix assertion in conversion from floating point types to ... by Alex Strelnikov <strel@google.com>

GitOrigin-RevId: c77facb7ba7c12d656e213e5d63d7dcd9d1f56ba
Change-Id: I7590561b04a986ffbec7139bb9c3aea319a3b975
This commit is contained in:
Abseil Team 2018-01-30 13:05:25 -08:00 committed by Jon Cohen
parent e5c6ee2d00
commit f4f238b09b
7 changed files with 34 additions and 19 deletions

View file

@ -1,4 +1,3 @@
## Abseil CMake build instructions ## Abseil CMake build instructions
@ -79,8 +78,3 @@ As of this writing, that pull request requires -DBUILD_TESTING=OFF as it doesn't
absl::synchronization absl::synchronization
absl::time absl::time
absl::utility absl::utility

View file

@ -62,7 +62,6 @@ list(APPEND BASE_INTERNAL_HEADERS
# absl_base main library # absl_base main library
list(APPEND BASE_SRC list(APPEND BASE_SRC
"internal/cycleclock.cc" "internal/cycleclock.cc"
"internal/exception_safety_testing.cc"
"internal/raw_logging.cc" "internal/raw_logging.cc"
"internal/spinlock.cc" "internal/spinlock.cc"
"internal/sysinfo.cc" "internal/sysinfo.cc"
@ -117,6 +116,28 @@ absl_library(
throw_delegate throw_delegate
) )
if(BUILD_TESTING)
# exception-safety testing library
set(EXCEPTION_SAFETY_TESTING_SRC "internal/exception_safety_testing.cc")
set(EXCEPTION_SAFETY_TESTING_PUBLIC_LIBRARIES
${ABSL_TEST_COMMON_LIBRARIES}
absl::base
absl::memory
absl::meta
absl::strings
absl::types
)
absl_library(
TARGET
absl_base_internal_exception_safety_testing
SOURCES
${EXCEPTION_SAFETY_TESTING_SRC}
PUBLIC_LIBRARIES
${EXCEPTION_SAFETY_TESTING_PUBLIC_LIBRARIES}
)
endif()
# dynamic_annotations library # dynamic_annotations library
set(DYNAMIC_ANNOTATIONS_SRC "dynamic_annotations.cc") set(DYNAMIC_ANNOTATIONS_SRC "dynamic_annotations.cc")

View file

@ -104,11 +104,15 @@ void DivModImpl(uint128 dividend, uint128 divisor, uint128* quotient_ret,
} }
template <typename T> template <typename T>
uint128 Initialize128FromFloat(T v) { uint128 MakeUint128FromFloat(T v) {
static_assert(std::is_floating_point<T>::value, "");
// Rounding behavior is towards zero, same as for built-in types. // Rounding behavior is towards zero, same as for built-in types.
// Undefined behavior if v is NaN or cannot fit into uint128. // Undefined behavior if v is NaN or cannot fit into uint128.
assert(!std::isnan(v) && v > -1 && v < std::ldexp(static_cast<T>(1), 128)); assert(std::isfinite(v) && v > -1 &&
(std::numeric_limits<T>::max_exponent <= 128 ||
v < std::ldexp(static_cast<T>(1), 128)));
if (v >= std::ldexp(static_cast<T>(1), 64)) { if (v >= std::ldexp(static_cast<T>(1), 64)) {
uint64_t hi = static_cast<uint64_t>(std::ldexp(v, -64)); uint64_t hi = static_cast<uint64_t>(std::ldexp(v, -64));
@ -120,9 +124,9 @@ uint128 Initialize128FromFloat(T v) {
} }
} // namespace } // namespace
uint128::uint128(float v) : uint128(Initialize128FromFloat(v)) {} uint128::uint128(float v) : uint128(MakeUint128FromFloat(v)) {}
uint128::uint128(double v) : uint128(Initialize128FromFloat(v)) {} uint128::uint128(double v) : uint128(MakeUint128FromFloat(v)) {}
uint128::uint128(long double v) : uint128(Initialize128FromFloat(v)) {} uint128::uint128(long double v) : uint128(MakeUint128FromFloat(v)) {}
uint128& uint128::operator/=(uint128 other) { uint128& uint128::operator/=(uint128 other) {
uint128 quotient = 0; uint128 quotient = 0;

View file

@ -233,8 +233,8 @@ class LOCKABLE Mutex {
// //
// Aliases for `Mutex::Lock()`, `Mutex::Unlock()`, and `Mutex::TryLock()`. // Aliases for `Mutex::Lock()`, `Mutex::Unlock()`, and `Mutex::TryLock()`.
// //
// Use the `Writer*()` versions of these method names when using complementary // These methods may be used (along with the complementary `Reader*()`
// `Reader*()` methods to distingish simple exclusive `Mutex` usage (`Lock()`, // methods) to distingish simple exclusive `Mutex` usage (`Lock()`,
// etc.) from reader/writer lock usage. // etc.) from reader/writer lock usage.
void WriterLock() EXCLUSIVE_LOCK_FUNCTION() { this->Lock(); } void WriterLock() EXCLUSIVE_LOCK_FUNCTION() { this->Lock(); }

View file

@ -75,4 +75,3 @@ absl_test(
${TIME_TEST_PUBLIC_LIBRARIES} ${TIME_TEST_PUBLIC_LIBRARIES}
) )

View file

@ -130,7 +130,7 @@ absl_test(
# test any_exception_safety_test # test any_exception_safety_test
set(ANY_EXCEPTION_SAFETY_TEST_SRC "any_exception_safety_test.cc") set(ANY_EXCEPTION_SAFETY_TEST_SRC "any_exception_safety_test.cc")
set(ANY_EXCEPTION_SAFETY_TEST_PUBLIC_LIBRARIES absl::any absl::base) set(ANY_EXCEPTION_SAFETY_TEST_PUBLIC_LIBRARIES absl::any absl::base absl::base_internal_exception_safety_testing)
absl_test( absl_test(
TARGET TARGET

View file

@ -43,6 +43,3 @@ absl_test(
PUBLIC_LIBRARIES PUBLIC_LIBRARIES
${UTILITY_TEST_PUBLIC_LIBRARIES} ${UTILITY_TEST_PUBLIC_LIBRARIES}
) )