Fixed compiler warnings and added gitignore

spcl · May 24, 2021 · cc749d5 · cc749d5
1 parent 140a78d
commit cc749d5
Show file tree

Hide file tree

Showing 7 changed files with 23 additions and 6 deletions.
diff --git a/.gitignore b/.gitignore
@@ -0,0 +1,3 @@
+/build*
+/cmake-build*
+venv
diff --git a/gms/algorithms/non_set_based/coloring/coloring_dense_sparse.h b/gms/algorithms/non_set_based/coloring/coloring_dense_sparse.h
@@ -205,7 +205,7 @@ class Coloring_Dense_Sparse {
 
             #pragma omp for schedule(dynamic,8)
             for (NodeId v = 0; v < n; v++) {
-    	        NodeId lastU = 0;
+    	        [[maybe_unused]] NodeId lastU = 0;
 
                 if(g.out_degree(v)>=friendNumber) {
                     auto N = g.out_neigh(v);

diff --git a/gms/algorithms/set_based/maximal_clique_enum/sub_graph/fast_roaring_sub_graph.h b/gms/algorithms/set_based/maximal_clique_enum/sub_graph/fast_roaring_sub_graph.h
@@ -27,14 +27,14 @@ class Fast_RoaringSubGraph
         int neighCount = subg.cardinality();
         this->mapping.reserve(neighCount);
         int counter = 0;
-        for (auto const w : cand)
+        for (NodeId const w : cand)
         {
             this->mapping.insert({w, counter});
             this->vertices[counter] = graph.out_neigh(w).intersect(subg);
             this->backReferences[counter] = {this->vertices[counter].intersect_count(cand), w};
             counter++;
         }
-        for (auto const w : fini)
+        for (NodeId const w : fini)
         {
             this->mapping.insert({w, counter});
             this->vertices[counter] = graph.out_neigh(w).intersect(cand);
@@ -49,14 +49,14 @@ class Fast_RoaringSubGraph
         int neighCount = subg.cardinality();
         this->mapping.reserve(neighCount);
         int counter = 0;
-        for (auto const w : cand)
+        for (NodeId const w : cand)
         {
             this->mapping.insert({w, counter});
             this->vertices[counter] = graph.neigh(w).intersect(subg);
             this->backReferences[counter] = {this->vertices[counter].intersect_count(cand), w};
             counter++;
         }
-        for (auto const w : fini)
+        for (NodeId const w : fini)
         {
             this->mapping.insert({w, counter});
             this->vertices[counter] = graph.neigh(w).intersect(cand);

diff --git a/gms/common/papi/papiw.h b/gms/common/papi/papiw.h
@@ -3,6 +3,8 @@
 
 #include "./papiw_util.h"
 
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wunused-function"
 /**
  * Papi Wrapper Highlevel Module
  * 
@@ -126,6 +128,7 @@ namespace PAPIW
 #endif
         }
 } // namespace PAPIW
+#pragma GCC diagnostic pop
 
 #ifdef NOPAPIW
 /* Provide PAPI Counter Macros, s.t. a program with deactivated PAPIW compiles nevertheless */

diff --git a/gms/third_party/gapbs/builder.h b/gms/third_party/gapbs/builder.h
@@ -756,6 +756,9 @@ VarintWordBasedGraph csrToVarintWordBased(const CSRGraphBase<NodeId_, DestID_, i
 		return VarintWordBasedGraph(num_nodes, num_edges, directed, new_offsets_out, new_offsets_in, new_adj_data_out, new_adj_data_in);
 	}
 
+//Ignore Compiler-Warning of 3rd party Code (GMS team)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wpessimizing-move"
 	template <class CGraph>
 	auto csrToCGraphGeneric(const CSRGraphBase<NodeId_, DestID_, invert> &csr_graph) {
         if constexpr (std::is_same_v<CGraph, CSRGraph>) {
@@ -780,7 +783,7 @@ VarintWordBasedGraph csrToVarintWordBased(const CSRGraphBase<NodeId_, DestID_, i
             static_assert(GMS::always_false<CGraph>, "class not supported");
         }
     }
-
+#pragma GCC diagnostic pop
 
 struct Tree {
 	Tree* left = 0; // left subtree

diff --git a/gms/third_party/robin_hood.h b/gms/third_party/robin_hood.h
@@ -1244,11 +1244,15 @@ class Table
             return *this;
         }
 
+//Ignore Compiler-Warning of 3rd party Code (GMS team)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wpessimizing-move"
         Iter operator++(int) noexcept {
             Iter tmp = *this;
             ++(*this);
             return std::move(tmp);
         }
+#pragma GCC diagnostic pop
 
         reference operator*() const {
             return **mKeyVals;

diff --git a/testing/cgraph.cpp b/testing/cgraph.cpp
@@ -34,7 +34,11 @@ CGraph load_CGraph(int)
 
     // TODO refactoring
     if constexpr (std::is_same_v<CGraph, CSRGraph>) {
+//Ignore Compiler-Warning
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wpessimizing-move"
         return std::move(g);
+#pragma GCC diagnostic pop
     } else {
         return builder.csrToCGraphGeneric<CGraph>(g);
     }