Revert "Preserve in-place result aliases and lifetimes during memory planning…"

This reverts commit 5ef2767.

Author: SS-JIA

exir/BUCK

@@ -168,19 +168,6 @@ fbcode_target(_kind = runtime.python_library,
     ],
 )
 
-fbcode_target(_kind = runtime.python_library,
-    name = "inplace_aliasing",
-    srcs = [
-        "inplace_aliasing.py",
-    ],
-    deps = [
-        ":error",
-        ":tensor",
-        "//caffe2:torch",
-        "//executorch/exir/operator:convert",
-    ],
-)
-
 fbcode_target(_kind = runtime.python_library,
     name = "memory_planning",
     srcs = [
@@ -190,7 +177,6 @@ fbcode_target(_kind = runtime.python_library,
         ":control_flow",
         ":delegate",
         ":error",
-        ":inplace_aliasing",
         ":memory",
         ":schema",
         ":tensor",

exir/emit/_emitter.py

@@ -638,8 +638,8 @@ def _emit_spec(self, spec: ValueSpec) -> _EmitterValue:
         """Given the provided spec constructs the corresponding EValue from it and then emits it.
 
         If `spec` was already emitted earlier (e.g., because two FX
-        nodes share the same TensorSpec object — typically because
-        `alias_inplace_result_specs` aliased an in-place
+        nodes share the same TensorSpec object — typically because the
+        planner's `_alias_inplace_result_specs` aliased an in-place
         op's result onto its mutated input), reuse the existing
         value_id. This keeps the invariant "one TensorSpec ↔ one
         Value" so downstream emit doesn't create duplicate Values for
@@ -2093,8 +2093,9 @@ def _is_buffer(node: Node, graph_signature: ExportGraphSignature) -> bool:
 
         # Populate spec2id_dict so downstream `_emit_spec` calls (e.g.,
         # for in-place op result FX nodes whose spec was aliased onto
-        # this placeholder's spec by `alias_inplace_result_specs`) reuse
-        # this placeholder's value_id rather than creating a new Value.
+        # this placeholder's spec by the planner's
+        # `_alias_inplace_result_specs`) reuse this placeholder's
+        # value_id rather than creating a new Value.
         if isinstance(spec, TensorSpec):
             self.emitter_state.spec2id_dict[spec] = value.id