Enable avx512

CMake/resolve_dependency_modules/hyperscan.cmake

@@ -34,19 +34,7 @@ set(CMAKE_CXX_STANDARD_BACKUP ${CMAKE_CXX_STANDARD})
 # C++ 17 is not supported.
 set(CMAKE_CXX_STANDARD 11)
 set(BUILD_EXAMPLES FALSE)
-#set(BUILD_AVX512 ON)
+set(BUILD_AVX512 ON)
 FetchContent_MakeAvailable(hyperscan)
 set(CMAKE_CXX_STANDARD ${CMAKE_CXX_STANDARD_BACKUP})
-# if(ICU_SOURCE)
-#   # empty var will cause a syntax error
-#   if(${ICU_SOURCE} STREQUAL "BUNDLED")
-#     # build re2 after icu so the files are available
-#     add_dependencies(re2 ICU ICU::uc)
-#   endif()
-# endif()
-
-# set(re2_LIBRARIES ${re2_BINARY_DIR}/libre2.a)
-# set(re2_INCLUDE_DIRS ${re2_SOURCE_DIR})
-
-# set(RE2_ROOT ${re2_BINARY_DIR})
-# set(re2_ROOT ${re2_BINARY_DIR})
+# TODO: Build and install colm, ragel before hyperscan.

velox/functions/lib/HyperscanFunctions.cpp

@@ -51,10 +51,7 @@ class MatchConstantPattern final : public exec::VectorFunction {
   }
 
   /**
-   * This is the function that will be called for each match that occurs. @a ctx
-   * is to allow you to have some application-specific state that you will get
-   * access to for each match. In our simple example we're just going to use it
-   * to pass in the pattern that was being searched for so we can print it out.
+   * Callback function for matching case.
    */
   static int eventHandler(
       unsigned int id,
@@ -135,4 +132,4 @@ std::shared_ptr<exec::VectorFunction> makeHyperscanMatch(
   VELOX_UNREACHABLE();
 }
 
-} // namespace facebook::velox::functions
+} // namespace facebook::velox::functions

velox/functions/lib/HyperscanFunctions.h

@@ -26,4 +26,4 @@ std::shared_ptr<exec::VectorFunction> makeHyperscanMatch(
     const std::vector<exec::VectorFunctionArg>& inputArgs,
     const core::QueryConfig& config);
 
-}
+}