X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fglsl%2Foptimize.cpp;h=32f7ce67959b388fa1bc2c2fb58ecc344a11bb87;hb=2280a09a286bbff2eb41783f8023863f320a3736;hp=596d7e20b071365850cb8d3e894b6eb0db910c1b;hpb=d9d7ab44d5f88c4537906e77c49db06d6bdc099c;p=libs%2Fgl.git

diff --git a/source/glsl/optimize.cpp b/source/glsl/optimize.cpp
index 596d7e20..32f7ce67 100644
--- a/source/glsl/optimize.cpp
+++ b/source/glsl/optimize.cpp
@@ -1,6 +1,7 @@
 #include <msp/core/raii.h>
 #include <msp/strings/format.h>
 #include "optimize.h"
+#include "reflect.h"
 
 using namespace std;
 
@@ -119,17 +120,13 @@ void InlineableFunctionLocator::visit(Return &ret)
 
 InlineContentInjector::InlineContentInjector():
 	source_func(0),
-	pass(DEPENDS)
+	pass(REFERENCED)
 { }
 
-const string &InlineContentInjector::apply(Stage &stage, FunctionDeclaration &target_func, Block &tgt_blk, const NodeList<Statement>::iterator &ins_pt, FunctionCall &call)
+string InlineContentInjector::apply(Stage &stage, FunctionDeclaration &target_func, Block &tgt_blk, const NodeList<Statement>::iterator &ins_pt, FunctionCall &call)
 {
 	source_func = call.declaration->definition;
 
-	// Collect all declarations the inlined function depends on.
-	pass = DEPENDS;
-	source_func->visit(*this);
-
 	/* Populate referenced_names from the target function so we can rename
 	variables from the inlined function that would conflict. */
 	pass = REFERENCED;
@@ -187,7 +184,7 @@ const string &InlineContentInjector::apply(Stage &stage, FunctionDeclaration &ta
 
 	tgt_blk.body.splice(ins_pt, staging_block.body);
 
-	NodeReorderer().apply(stage, target_func, dependencies);
+	NodeReorderer().apply(stage, target_func, DependencyCollector().apply(*source_func));
 
 	return r_result_name;
 }
@@ -200,31 +197,19 @@ void InlineContentInjector::visit(VariableReference &var)
 		if(i!=staging_block.variables.end())
 			var.name = i->second->name;
 	}
-	else if(pass==DEPENDS && var.declaration)
-	{
-		dependencies.insert(var.declaration);
-		var.declaration->visit(*this);
-	}
 	else if(pass==REFERENCED)
 		referenced_names.insert(var.name);
 }
 
 void InlineContentInjector::visit(InterfaceBlockReference &iface)
 {
-	if(pass==DEPENDS && iface.declaration)
-	{
-		dependencies.insert(iface.declaration);
-		iface.declaration->visit(*this);
-	}
-	else if(pass==REFERENCED)
+	if(pass==REFERENCED)
 		referenced_names.insert(iface.name);
 }
 
 void InlineContentInjector::visit(FunctionCall &call)
 {
-	if(pass==DEPENDS && call.declaration)
-		dependencies.insert(call.declaration);
-	else if(pass==REFERENCED)
+	if(pass==REFERENCED)
 		referenced_names.insert(call.name);
 	TraversingVisitor::visit(call);
 }
@@ -249,11 +234,6 @@ void InlineContentInjector::visit(VariableDeclaration &var)
 			}
 		}
 	}
-	else if(pass==DEPENDS && var.type_declaration)
-	{
-		dependencies.insert(var.type_declaration);
-		var.type_declaration->visit(*this);
-	}
 	else if(pass==REFERENCED)
 		referenced_names.insert(var.type);
 }
@@ -368,15 +348,6 @@ void FunctionInliner::visit(Iteration &iter)
 }
 
 
-ExpressionInliner::ExpressionInfo::ExpressionInfo():
-	expression(0),
-	assign_scope(0),
-	inline_point(0),
-	trivial(false),
-	available(true)
-{ }
-
-
 ExpressionInliner::ExpressionInliner():
 	r_ref_info(0),
 	r_any_inlined(false),
@@ -1026,10 +997,6 @@ bool UnusedVariableRemover::apply(Stage &s)
 		if(i->used_by.empty())
 			unused_nodes.insert(i->node);
 
-	for(map<string, InterfaceBlock *>::const_iterator i=s.interface_blocks.begin(); i!=s.interface_blocks.end(); ++i)
-		if(i->second->instance_name.empty())
-			unused_nodes.insert(i->second);
-
 	for(BlockVariableMap::const_iterator i=variables.begin(); i!=variables.end(); ++i)
 	{
 		if(i->second.output)
@@ -1182,16 +1149,8 @@ void UnusedVariableRemover::visit(VariableDeclaration &var)
 
 void UnusedVariableRemover::visit(InterfaceBlock &iface)
 {
-	if(iface.instance_name.empty())
-	{
-		SetForScope<InterfaceBlock *> set_block(interface_block, &iface);
-		iface.struct_declaration->members.visit(*this);
-	}
-	else
-	{
-		VariableInfo &var_info = variables[&iface];
-		var_info.output = (iface.interface=="out" && (iface.linked_block || !iface.block_name.compare(0, 3, "gl_")));
-	}
+	VariableInfo &var_info = variables[&iface];
+	var_info.output = (iface.interface=="out" && (iface.linked_block || !iface.block_name.compare(0, 3, "gl_")));
 }
 
 void UnusedVariableRemover::merge_variables(const BlockVariableMap &other_vars)