[MISC] Add miss Type2Str and remove compile warnings

include/tvm/relay/expr_functor.h

@@ -232,6 +232,8 @@ class ExprMutator : public ::tvm::relay::ExprFunctor<Expr(const Expr&)> {
  */
 class MixedModeVisitor : public ::tvm::relay::ExprVisitor {
  public:
+  using ::tvm::relay::ExprFunctor<void(const Expr& n)>::VisitExpr_;
+
   /*! \brief The constructor of MixedModeVisitor
    *  \param visit_limit The number of times to allow visitation to a node. Usually 1, ocassionally
    * higher (i.e., 2 for dead code elimiation), limited to 10 as a sanity check.
@@ -277,6 +279,8 @@ class MixedModeVisitor : public ::tvm::relay::ExprVisitor {
  */
 class MixedModeMutator : public ::tvm::relay::ExprMutator {
  public:
+  using ::tvm::relay::ExprFunctor<Expr(const Expr&)>::VisitExpr_;
+
   MixedModeMutator(bool pre = false) : pre_{pre} {};
   Expr VisitExpr(const Expr& expr) final;
 

include/tvm/runtime/container/shape_tuple.h

@@ -133,8 +133,8 @@ class ShapeTuple : public ObjectRef {
    * \return the i-th element.
    */
   index_type operator[](size_t idx) const {
-    ICHECK(0 <= idx && idx < this->size())
-        << "IndexError: indexing " << idx << " on an array of size " << this->size();
+    ICHECK(idx < this->size()) << "IndexError: indexing " << idx << " on an array of size "
+                               << this->size();
     return this->data()[idx];
   }
 

include/tvm/runtime/packed_func.h

@@ -1448,6 +1448,10 @@ struct Type2Str<DataType> {
   static std::string v() { return "DataType"; }
 };
 template <>
+struct Type2Str<DLDataType> {
+  static std::string v() { return "DLDataType"; }
+};
+template <>
 struct Type2Str<TVMRetValue> {
   static std::string v() { return "TVMRetValue"; }
 };