Commit 2b8cb090 authored by Sara Golemon's avatar Sara Golemon Committed by facebook-github-bot-1

Limit declaration of GroupVarint tables to SSE >= 2

Summary: __m128i is only available in SSE 2 and later.
We technically only use it in folly/GroupVarint.h for SSE 3+,
but that's due to various other intrinsics and we could
potentially lower the requirement in certain contexts,
so do the right thing in the generated table.
Revisit the uses in folly/GroupVarint.h later.

Reviewed By: @yfeldblum

Differential Revision: D2445798
parent 621cc26e
......@@ -54,7 +54,7 @@ def generate(f):
f.write("""
#include <folly/Portability.h>
#if FOLLY_X64 || defined(__i386__)
#if (FOLLY_X64 || defined(__i386__)) && (FOLLY_SSE >= 2)
#include <stdint.h>
#include <x86intrin.h>
......@@ -98,7 +98,7 @@ extern const __m128i groupVarintSSEMasks[] = {
} // namespace detail
} // namespace folly
#endif /* FOLLY_X64 || defined(__i386__) */
#endif /* (FOLLY_X64 || defined(__i386__)) && (FOLLY_SSE >= 2) */
""")
def main():
......
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