lemire/FastPFor

CMake build needs to be updated

Closed this issue · 4 comments

lemire commented

There are reports that cmake no longer works.

searchivarius commented

Works for me (cmake 2.8.7 Linux, g++4.7)
However, it seems that varintdecode.c misses a header. I added stdint.h, not sure if this will work on windows/mac though:

--- a/src/varintdecode.c
+++ b/src/varintdecode.c
@@ -2,6 +2,7 @@
  * (c) Indeed.com
  * All rights reserved (as of March 2014)
  */
+#include <stdint.h>

 #include <x86intrin.h>
searchivarius commented

Sorry, g++ 4.8.1

lemire commented

@searchivarius I have added the missing header.

lemire commented

Closing. Reopen if you disagree.