From 469b25ba1efe656a12d5fbdf80d4f137c6bf92e7 Mon Sep 17 00:00:00 2001
From: wrongtest <wrongtest0@gmail.com>
Date: Sun, 16 Jan 2022 04:55:11 +0800
Subject: [PATCH] remove clang compile warnings (#9942)

---
 src/relay/ir/indexed_graph.cc                |  2 ++
 src/tir/analysis/device_constraint_utils.cc  | 16 ++++++++--------
 src/tir/schedule/primitive/block_annotate.cc |  6 +++---
 src/tir/transforms/unify_thread_binding.cc   |  2 +-
 4 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/src/relay/ir/indexed_graph.cc b/src/relay/ir/indexed_graph.cc
index df648abba57c7..4efe57b491db0 100644
--- a/src/relay/ir/indexed_graph.cc
+++ b/src/relay/ir/indexed_graph.cc
@@ -45,6 +45,8 @@ IndexedGraph<Expr> CreateIndexedGraph(const Expr& expr) {
     }
 
    protected:
+    using MixedModeVisitor::VisitExpr_;
+
     void VisitLeaf(const Expr& expr) override {
       MixedModeVisitor::VisitLeaf(expr);
       auto node = std::make_shared<IndexedGraph<Expr>::Node>(expr, index_++);
diff --git a/src/tir/analysis/device_constraint_utils.cc b/src/tir/analysis/device_constraint_utils.cc
index ae2b9500649e0..26cf66c4d4c01 100644
--- a/src/tir/analysis/device_constraint_utils.cc
+++ b/src/tir/analysis/device_constraint_utils.cc
@@ -255,7 +255,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
     if (!new_buffer_var.same_as(new_load->buffer_var)) {
       return Load(load_node->dtype, new_buffer_var, load_node->index, load_node->predicate);
     }
-    return new_load;
+    return std::move(new_load);
   }
 
   PrimExpr VisitExpr_(const BufferLoadNode* buffer_load_node) final {
@@ -265,7 +265,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
     if (!new_buffer.same_as(new_buffer_load->buffer)) {
       return BufferLoad(new_buffer, new_buffer_load->indices, new_buffer_load->span);
     }
-    return new_buffer_load;
+    return std::move(new_buffer_load);
   }
 
   Stmt VisitStmt_(const LetStmtNode* let_stmt_node) final {
@@ -284,7 +284,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
                         new_attr_stmt->body);
       }
     }
-    return new_attr_stmt;
+    return std::move(new_attr_stmt);
   }
 
   // ForNode default ok since loop_var never of PointerType
@@ -302,7 +302,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
     if (!new_buffer_var.same_as(new_store->buffer_var)) {
       Store(new_buffer_var, new_store->value, new_store->index, new_store->predicate);
     }
-    return new_store;
+    return std::move(new_store);
   }
 
   Stmt VisitStmt_(const BufferStoreNode* buffer_store_node) final {
@@ -313,7 +313,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
       return BufferStore(new_buffer, new_buffer_store->value, new_buffer_store->indices,
                          new_buffer_store->span);
     }
-    return new_buffer_store;
+    return std::move(new_buffer_store);
   }
 
   Stmt VisitStmt_(const BufferRealizeNode* buffer_realize_node) final {
@@ -324,7 +324,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
       return BufferRealize(new_buffer, new_buffer_realize->bounds, new_buffer_realize->condition,
                            new_buffer_realize->body, new_buffer_realize->span);
     }
-    return new_buffer_realize;
+    return std::move(new_buffer_realize);
   }
 
   // IfThenElseNode default ok
@@ -338,7 +338,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
     if (!new_buffer.same_as(new_prefetch->buffer)) {
       return Prefetch(new_buffer, prefetch_node->bounds, prefetch_node->span);
     }
-    return new_prefetch;
+    return std::move(new_prefetch);
   }
 
   // SeqStmtNode default ok
@@ -390,7 +390,7 @@ class ApplyDeviceConstraintsMutator : public StmtExprMutator {
                    new_block->name_hint, new_block->body, new_block->init, new_block->alloc_buffers,
                    std::move(new_match_buffers), new_block->annotations, new_block->span);
     }
-    return new_block;
+    return std::move(new_block);
   }
 
   // BlockRealizeNode default ok
diff --git a/src/tir/schedule/primitive/block_annotate.cc b/src/tir/schedule/primitive/block_annotate.cc
index 181e5a6cfa697..418e770a5c932 100644
--- a/src/tir/schedule/primitive/block_annotate.cc
+++ b/src/tir/schedule/primitive/block_annotate.cc
@@ -278,7 +278,7 @@ class StorageScopeMutator : StmtExprMutator {
       ptr->buffer = it->second;
       return PrimExpr(ptr);
     } else {
-      return res;
+      return std::move(res);
     }
   }
 
@@ -291,7 +291,7 @@ class StorageScopeMutator : StmtExprMutator {
       ptr->buffer = it->second;
       return Stmt(ptr);
     } else {
-      return res;
+      return std::move(res);
     }
   }
 
@@ -348,7 +348,7 @@ class StorageScopeMutator : StmtExprMutator {
 
       Block new_block(n);
       block_sref_reuse_->Set(GetRef<Block>(block), new_block);
-      return new_block;
+      return std::move(new_block);
     }
   }
 
diff --git a/src/tir/transforms/unify_thread_binding.cc b/src/tir/transforms/unify_thread_binding.cc
index 9c1aab6344961..d9b5f529a35c1 100644
--- a/src/tir/transforms/unify_thread_binding.cc
+++ b/src/tir/transforms/unify_thread_binding.cc
@@ -66,7 +66,7 @@ class ThreadBindingUnifier : public StmtExprMutator {
     }
     For new_loop = Downcast<For>(stmt);
     new_loop.CopyOnWrite()->annotations = std::move(annotations);
-    return new_loop;
+    return std::move(new_loop);
   }
 
   template <typename Node>