Fix compiler warnings

llvm · Oct 4, 2024 · 09f07f5 · 09f07f5
1 parent a156c90
commit 09f07f5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/Dialect/DC/Transforms/DCPrintDot.cpp b/lib/Dialect/DC/Transforms/DCPrintDot.cpp
@@ -136,7 +136,7 @@ valueToName(const SmallVector<mlir::Value> &values,
   SmallVector<std::pair<mlir::Value, std::string>> res;
   for (auto [i, v] : llvm::enumerate(values)) {
     auto found = false;
-    for (const auto [key, value] : currentMap) {
+    for (const auto &[key, value] : currentMap) {
       if (v == key) {
         res.push_back({v, value});
         found = true;

diff --git a/unittests/Dialect/OM/Evaluator/EvaluatorTests.cpp b/unittests/Dialect/OM/Evaluator/EvaluatorTests.cpp
@@ -1055,7 +1055,7 @@ TEST(EvaluatorTests, ListConcat) {
   auto finalList =
       llvm::cast<evaluator::ListValue>(fieldValue.get())->getElements();
 
-  ASSERT_EQ(3, finalList.size());
+  ASSERT_EQ(3U, finalList.size());
 
   ASSERT_EQ(0, llvm::cast<evaluator::AttributeValue>(finalList[0].get())
                    ->getAs<circt::om::IntegerAttr>()
@@ -1114,7 +1114,7 @@ TEST(EvaluatorTests, ListConcatField) {
   auto finalList =
       llvm::cast<evaluator::ListValue>(fieldValue.get())->getElements();
 
-  ASSERT_EQ(3, finalList.size());
+  ASSERT_EQ(3U, finalList.size());
 
   ASSERT_EQ(0, llvm::cast<evaluator::AttributeValue>(finalList[0].get())
                    ->getAs<circt::om::IntegerAttr>()