[RUNTIME] Quick fix PackedFunc String passing (#5266)

include/tvm/runtime/packed_func.h

@@ -513,9 +513,12 @@ class TVMArgValue : public TVMPODValue_ {
     }
   }
   operator tvm::runtime::String() const {
-    // directly use the std::string constructor for now.
+    if (IsObjectRef<tvm::runtime::String>()) {
+      return AsObjectRef<tvm::runtime::String>();
+    } else {
       return tvm::runtime::String(operator std::string());
     }
+  }
   operator DLDataType() const {
     if (type_code_ == kTVMStr) {
       return String2DLDataType(operator std::string());
@@ -605,9 +608,12 @@ class TVMRetValue : public TVMPODValue_ {
     return *ptr<std::string>();
   }
   operator tvm::runtime::String() const {
-    // directly use the std::string constructor for now.
+    if (IsObjectRef<tvm::runtime::String>()) {
+      return AsObjectRef<tvm::runtime::String>();
+    } else {
       return tvm::runtime::String(operator std::string());
     }
+  }
   operator DLDataType() const {
     if (type_code_ == kTVMStr) {
       return String2DLDataType(operator std::string());

tests/cpp/packed_func_test.cc

@@ -95,6 +95,12 @@ TEST(PackedFunc, str) {
       CHECK(y == "hello");
       *rv = x;
     })("hello");
+
+  PackedFunc([&](TVMArgs args, TVMRetValue* rv) {
+      CHECK(args.num_args == 1);
+      runtime::String s = args[0];
+      CHECK(s == "hello");
+  })(runtime::String("hello"));
 }