remove clang-specific logic to ignore compiler warnings

openfheorg · Jun 27, 2024 · 0d78237 · 0d78237
1 parent 554c619
commit 0d78237
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -154,10 +154,6 @@ if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")
     message (STATUS "BUILTIN_INFO_AVAILABLE is defined")
 endif()
 
-if("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang")
-    set(IGNORE_WARNINGS "${IGNORE_WARNINGS} -Wno-unused-private-field -Wno-shift-op-parentheses")
-endif()
-
 if( WITH_NATIVEOPT )
     set (NATIVE_OPT "-march=native")
 else()

diff --git a/src/binfhe/lib/binfhe-base-scheme.cpp b/src/binfhe/lib/binfhe-base-scheme.cpp
@@ -309,7 +309,7 @@ LWECiphertext BinFHEScheme::EvalFunc(const std::shared_ptr<BinFHECryptoParams>&
 LWECiphertext BinFHEScheme::EvalFloor(const std::shared_ptr<BinFHECryptoParams>& params, const RingGSWBTKey& EK,
                                       ConstLWECiphertext& ct, const NativeInteger& beta, uint32_t roundbits) const {
     const auto& LWEParams = params->GetLWEParams();
-    NativeInteger q{roundbits == 0 ? LWEParams->Getq() : beta * (1 << roundbits + 1)};
+    NativeInteger q{roundbits == 0 ? LWEParams->Getq() : beta * (1 << (roundbits + 1))};
     NativeInteger mod{ct->GetModulus()};
 
     auto ct1 = std::make_shared<LWECiphertextImpl>(*ct);

diff --git a/src/pke/unittest/utbgvrns/UnitTestBGVrnsAdvancedSHE.cpp b/src/pke/unittest/utbgvrns/UnitTestBGVrnsAdvancedSHE.cpp
@@ -119,8 +119,7 @@ static std::vector<TEST_CASE_UTBGVRNS_SHEADVANCED> testCasesUTBGVRNS_SHEADVANCED
 // clang-format on
 //===========================================================================================================
 class UTBGVRNS_SHEADVANCED : public ::testing::TestWithParam<TEST_CASE_UTBGVRNS_SHEADVANCED> {
-    using Element    = DCRTPoly;
-    const double eps = EPSILON;
+    using Element = DCRTPoly;
 
 protected:
     void SetUp() {}