fix bug, when dim of aten::size.int(Tensor self, int dim) -> (int) is negative

Signed-off-by: Ruoqian Guo <[email protected]>

Author: ruoqianguo

core/conversion/evaluators/aten.cpp

@@ -180,9 +180,16 @@ auto aten_registrations TRTORCH_UNUSED =
                         auto dim = args.at(n->input(1)).unwrapToInt();
                         if (tensor_var.isITensor()) {
                           auto tensor = tensor_var.ITensor();
-                          return util::toVec(tensor->getDimensions())[dim];
+                          auto dims = util::toVec(tensor->getDimensions());
+                          auto nbDims = tensor->getDimensions().nbDims;
+                          if (dim < 0)
+                            dim += nbDims;
+                          return dims[dim];
                         } else {
                           auto tensor = tensor_var.unwrapToTensor();
+                          auto nbDims = tensor.sizes().size();
+                          if (dim < 0)
+                            dim += nbDims;
                           return tensor.sizes()[dim];
                         }
                       }

tests/core/conversion/evaluators/test_aten_evaluators.cpp

@@ -75,4 +75,29 @@ TEST(Evaluators, ZerosDataTypeEvaluatesCorrectly) {
   auto trt_results = trtorch::tests::util::EvaluateGraph(g->block(), {in});
 
   ASSERT_TRUE(at::equal(jit_results[0].toTensor().to(at::kCUDA), trt_results[0].toTensor()));
+}
+
+TEST(Evaluators, SizeConvertsCorrectly) {
+  const auto graph = R"IR(
+      graph(%0 : Tensor):
+        %1 : int = prim::Constant[value=-1]()
+        %2 : int = prim::Constant[value=-2]()
+        %3 : int = aten::size(%0, %1)
+        %4 : int = aten::size(%0, %2)
+        %5 : int[] = prim::ListConstruct(%3, %4)
+        %6 : Tensor = aten::view(%0, %5)
+        return (%6))IR";
+
+  auto g = std::make_shared<torch::jit::Graph>();
+  torch::jit::parseIR(graph, &*g);
+
+  auto in = at::randint(1, 10, {3, 3}, {at::kCUDA});
+
+  auto params = trtorch::core::conversion::get_named_params(g->inputs(), {});
+  auto jit_results = trtorch::tests::util::RunGraph(g, params, {in});
+
+  params = trtorch::core::conversion::get_named_params(g->inputs(), {});
+  auto trt_results = trtorch::tests::util::RunGraphEngine(g, params, {in});
+
+  ASSERT_TRUE(trtorch::tests::util::almostEqual(jit_results[0], trt_results[0].reshape_as(jit_results[0]), 2e-6));
 }