From 183c67b022da945237e65947117f61fcdf76e14b Mon Sep 17 00:00:00 2001 From: GelilaSeifu Date: Fri, 22 Oct 2021 15:20:52 -0700 Subject: [PATCH 1/2] Update to HEXL v1.2.2 --- CMakeLists.txt | 2 +- README.md | 2 +- cmake/ExternalIntelHEXL.cmake | 2 +- native/src/seal/util/polyarithsmallmod.cpp | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 80ece2e1..b042e35a 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -223,7 +223,7 @@ if(SEAL_USE_INTEL_HEXL) message(STATUS "Intel HEXL: download ...") seal_fetch_thirdparty_content(ExternalIntelHEXL) else() - find_package(HEXL 1.2.1) + find_package(HEXL 1.2.2) if (NOT TARGET HEXL::hexl) message(FATAL_ERROR "Intel HEXL: not found") endif() diff --git a/README.md b/README.md index 2459e055..8dd3d5b4 100644 --- a/README.md +++ b/README.md @@ -116,7 +116,7 @@ The optional dependencies and their tested versions (other versions may work as | Optional dependency | Tested version | Use | | ------------------------------------------------------ | -------------- | ------------------------------------------------ | -| [Intel HEXL](https://github.com/intel/hexl) | 1.2.1 | Acceleration of low-level kernels | +| [Intel HEXL](https://github.com/intel/hexl) | 1.2.2 | Acceleration of low-level kernels | | [Microsoft GSL](https://github.com/microsoft/GSL) | 3.1.0 | API extensions | | [ZLIB](https://github.com/madler/zlib) | 1.2.11 | Compressed serialization | | [Zstandard](https://github.com/facebook/zstd) | 1.4.5 | Compressed serialization (much faster than ZLIB) | diff --git a/cmake/ExternalIntelHEXL.cmake b/cmake/ExternalIntelHEXL.cmake index 410f141b..1feacac9 100644 --- a/cmake/ExternalIntelHEXL.cmake +++ b/cmake/ExternalIntelHEXL.cmake @@ -5,7 +5,7 @@ FetchContent_Declare( hexl PREFIX hexl GIT_REPOSITORY https://github.com/intel/hexl - GIT_TAG f976b7d820a605973819863cb201a0e1a4ecf96b # beyond 1.2.1 + GIT_TAG 1.2.2 ) FetchContent_GetProperties(hexl) diff --git a/native/src/seal/util/polyarithsmallmod.cpp b/native/src/seal/util/polyarithsmallmod.cpp index 29f63e9b..e3f0afca 100644 --- a/native/src/seal/util/polyarithsmallmod.cpp +++ b/native/src/seal/util/polyarithsmallmod.cpp @@ -33,7 +33,7 @@ namespace seal #endif #ifdef SEAL_USE_INTEL_HEXL - intel::hexl::EltwiseReduceMod(result, poly, coeff_count, modulus.value(), 0, 1); + intel::hexl::EltwiseReduceMod(result, poly, coeff_count, modulus.value(), modulus.value(), 1); #else SEAL_ITERATE( iter(poly, result), coeff_count, [&](auto I) { get<1>(I) = barrett_reduce_64(get<0>(I), modulus); }); From e21a7517469c269f99757924de9b648c98033faa Mon Sep 17 00:00:00 2001 From: GelilaSeifu Date: Fri, 22 Oct 2021 16:04:35 -0700 Subject: [PATCH 2/2] change branch to commit hash --- cmake/ExternalIntelHEXL.cmake | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/ExternalIntelHEXL.cmake b/cmake/ExternalIntelHEXL.cmake index 1feacac9..db1b4fe3 100644 --- a/cmake/ExternalIntelHEXL.cmake +++ b/cmake/ExternalIntelHEXL.cmake @@ -5,7 +5,7 @@ FetchContent_Declare( hexl PREFIX hexl GIT_REPOSITORY https://github.com/intel/hexl - GIT_TAG 1.2.2 + GIT_TAG 343acab # 1.2.2 ) FetchContent_GetProperties(hexl)