Commit a251c84f authored by Yedidya Feldblum's avatar Yedidya Feldblum Committed by Facebook Github Bot 9

Remove the glog header include from Assume.h

Summary:
[Folly] Remove the glog header include from `Assume.h`.

Better to avoid unnecessary includes.

Reviewed By: Orvid

Differential Revision: D3652651

fbshipit-source-id: 3fa6256e9571539c692b9c50c1c215b31eef394a
parent fcd39524
/*
* Copyright 2016 Facebook, Inc.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
#include <folly/Assume.h>
#include <glog/logging.h>
namespace folly {
namespace detail {
void assume_check(bool cond) {
CHECK(cond) << "compiler-hint assumption fails at runtime";
}
}
}
......@@ -19,10 +19,15 @@
#include <cstdlib>
#include <folly/Portability.h>
#include <glog/logging.h>
namespace folly {
namespace detail {
extern void assume_check(bool cond);
}
/**
* Inform the compiler that the argument can be assumed true. It is
* undefined behavior if the argument is not actually true, so use
......@@ -34,17 +39,19 @@ namespace folly {
*/
FOLLY_ALWAYS_INLINE void assume(bool cond) {
#ifndef NDEBUG
DCHECK(cond);
#elif defined(__clang__) // Must go first because Clang also defines __GNUC__.
__builtin_assume(cond);
if (kIsDebug) {
detail::assume_check(cond);
} else {
#if defined(__clang__) // Must go first because Clang also defines __GNUC__.
__builtin_assume(cond);
#elif defined(__GNUC__)
if (!cond) { __builtin_unreachable(); }
if (!cond) { __builtin_unreachable(); }
#elif defined(_MSC_VER)
__assume(cond);
__assume(cond);
#else
// Do nothing.
// Do nothing.
#endif
}
}
[[noreturn]] FOLLY_ALWAYS_INLINE void assume_unreachable() {
......
......@@ -369,6 +369,7 @@ libfollybase_la_SOURCES = \
Unicode.cpp
libfolly_la_SOURCES = \
Assume.cpp \
Bits.cpp \
Checksum.cpp \
ClockGettimeWrappers.cpp \
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment