[Mesa-dev] [PATCH 5/9] glsl: Skip processing the first function's body in do_dead_functions().

Eric Anholt eric at anholt.net
Fri Mar 11 16:06:49 PST 2011


It can't call anything, so there's no point.
---
 src/glsl/opt_dead_functions.cpp |   11 ++++++++++-
 1 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/src/glsl/opt_dead_functions.cpp b/src/glsl/opt_dead_functions.cpp
index 51c77e3..7c64c61 100644
--- a/src/glsl/opt_dead_functions.cpp
+++ b/src/glsl/opt_dead_functions.cpp
@@ -50,6 +50,7 @@ public:
    ir_dead_functions_visitor()
    {
       this->mem_ctx = ralloc_context(NULL);
+      this->seen_another_function_signature = false;
    }
 
    ~ir_dead_functions_visitor()
@@ -64,6 +65,8 @@ public:
 
    bool (*predicate)(ir_instruction *ir);
 
+   bool seen_another_function_signature;
+
    /* List of signature_entry */
    exec_list signature_list;
    void *mem_ctx;
@@ -94,7 +97,13 @@ ir_dead_functions_visitor::visit_enter(ir_function_signature *ir)
       entry->used = true;
    }
 
-
+   /* If this is the first signature to look at, no need to descend to see
+    * if it has calls to another function signature.
+    */
+   if (!this->seen_another_function_signature) {
+      this->seen_another_function_signature = true;
+      return visit_continue_with_parent;
+   }
 
    return visit_continue;
 }
-- 
1.7.4.1



More information about the mesa-dev mailing list