Merge branch 'main' into revert-1285-add_index_bool

Author: justinchuby

onnxscript/function_libs/torch_lib/graph_building.py

@@ -791,6 +791,8 @@ def _override_with_symbolic_value_info_proto(self, onnx_model: onnx.ModelProto):
                 continue
             for i, input_info in enumerate(onnx_model.graph.input):
                 if input_info.name == input.debugName():
+                    # See NOTE: _C.Value re-naming.
+                    value_info.name = input_info.name
                     onnx_model.graph.input.insert(i, value_info)
                     onnx_model.graph.input.remove(input_info)
                     break
@@ -804,6 +806,8 @@ def _override_with_symbolic_value_info_proto(self, onnx_model: onnx.ModelProto):
                 continue
             for i, output_info in enumerate(onnx_model.graph.output):
                 if output_info.name == output.debugName():
+                    # See NOTE: _C.Value re-naming.
+                    value_info.name = output_info.name
                     onnx_model.graph.output.insert(i, value_info)
                     onnx_model.graph.output.remove(output_info)
                     break
@@ -920,6 +924,12 @@ def generate_maingraph_value_info_proto(self) -> Mapping[str, onnx.ValueInfoProt
         for torch_value, tensor in self._value_to_tensor.items():
             if (value_info := tensor.value_info()) is None:
                 continue
+            # NOTE: _C.Value re-naming.
+            # _C.Value's debugName is unstable.
+            # When duplicated names are encountered, all names involved are updated by
+            # TorchScript naming strategy. Hence the previous name stored in value_info
+            # can be outdated.
+            value_info.name = torch_value.debugName()
             named_value_info[torch_value.debugName()] = value_info
         return named_value_info