Commit 49efc3ab authored by Xiao Shi's avatar Xiao Shi Committed by Facebook Github Bot

fix HAVE_INT128_TRAITS cmake test

Summary:
C++17 defined integer traits for `__int128` as well as a (trivial) std::hash,
this diff adds the check in `FOLLY_SUPPLY_MISSING_INT128_TRAITS`.

There was an unintended linebreak in the cmake test which rendered it non
functional. This diff fixes it.

Original issue:
https://github.com/facebook/folly/commit/b2b96f59c57cd7aba686cc4f61ceba0314cdeb72#comments

Reviewed By: yfeldblum

Differential Revision: D7138006

fbshipit-source-id: ca1841b9ca847f1f6570b58e95380f50fe092345
parent 8e458f98
...@@ -131,8 +131,8 @@ if (NOT INT128_SIZE STREQUAL "") ...@@ -131,8 +131,8 @@ if (NOT INT128_SIZE STREQUAL "")
__int128>::value, __int128>::value,
\"signed form of 'unsigned __uint128' must be '__int128'.\"); \"signed form of 'unsigned __uint128' must be '__int128'.\");
static_assert( static_assert(
sizeof(::std::hash<__int128>{}(0)) > 0, \"std::hash<__int128> is sizeof(::std::hash<__int128>{}(0)) > 0, \
disabled.\"); \"std::hash<__int128> is disabled.\");
int main() { return 0; }" int main() { return 0; }"
HAVE_INT128_TRAITS HAVE_INT128_TRAITS
) )
......
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