[ET-VK][runtime] Inline resize update checks

Pull Request resolved: pytorch#22498

Inline direct generation-stamp lookup while keeping rare nested ValueList traversal out of line.

Authored with Codex.
ghstack-source-id: 423933067
@exported-using-ghexport

Differential Revision: [D118543873](https://our.internmc.facebook.com/intern/diff/D118543873/)

Author: SS-JIA

backends/vulkan/runtime/graph/ComputeGraph.cpp

@@ -242,26 +242,13 @@ utils::StorageType ComputeGraph::suggested_storage_type() {
   return utils::kTexture3D;
 }
 
-bool ComputeGraph::was_value_updated(const ValueRef idx) const noexcept {
-  if (!is_valid_value_idx(idx)) {
-    return false;
-  }
-
-  const size_t value_idx = static_cast<size_t>(idx);
-  if (value_idx < value_update_generations_.size() &&
-      value_update_generations_[value_idx] == current_update_generation_) {
-    return true;
-  }
-
-  if (val_is_value_list(idx)) {
-    const auto& value_list = values_.at(idx).toConstValueList();
-    for (const auto& nested_idx : value_list) {
-      if (was_value_updated(nested_idx)) {
-        return true;
-      }
+bool ComputeGraph::was_value_list_updated(const ValueRef idx) const noexcept {
+  const auto& value_list = values_[static_cast<size_t>(idx)].toConstValueList();
+  for (const auto nested_idx : value_list) {
+    if (was_value_updated(nested_idx)) {
+      return true;
     }
   }
-
   return false;
 }
 

backends/vulkan/runtime/graph/ComputeGraph.h

@@ -712,6 +712,7 @@ class ComputeGraph final {
 
  private:
   void check_no_active_value_ptrs();
+  bool was_value_list_updated(const ValueRef idx) const noexcept;
 
  public:
   /*
@@ -1174,7 +1175,22 @@ class ComputeGraph final {
 
   // Check if a specific ValueRef (or ValueList) was updated, with recursive
   // handling
-  bool was_value_updated(const ValueRef idx) const noexcept;
+  inline bool was_value_updated(const ValueRef idx) const noexcept {
+    if (idx < 0) {
+      return false;
+    }
+
+    const size_t value_idx = static_cast<size_t>(idx);
+    if (value_idx >= values_.size()) {
+      return false;
+    }
+    if (value_idx < value_update_generations_.size() &&
+        value_update_generations_[value_idx] == current_update_generation_) {
+      return true;
+    }
+
+    return values_[value_idx].isValueList() && was_value_list_updated(idx);
+  }
 
   // Set the flag to indicate that re-encoding is required
   inline void set_requires_reencode() noexcept {