X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fglsl%2Foptimize.cpp;h=6ac84b5b0247be7c48ffe3339ba288c1d047ab3c;hb=d8bdf61007978e2c3670a22a58e2f105e8347537;hp=7d605f1f12bc755812a3252172828a84d3720777;hpb=4790c6a06072814b21f5b3f24b53c6936915308d;p=libs%2Fgl.git

diff --git a/source/glsl/optimize.cpp b/source/glsl/optimize.cpp
index 7d605f1f..6ac84b5b 100644
--- a/source/glsl/optimize.cpp
+++ b/source/glsl/optimize.cpp
@@ -91,7 +91,6 @@ const string &InlineContentInjector::apply(Stage &stage, FunctionDeclaration &ta
 	pass = INLINE;
 	staging_block.parent = &tgt_blk;
 	staging_block.variables.clear();
-	remap_prefix = source_func->name;
 
 	std::vector<RefPtr<VariableDeclaration> > params;
 	params.reserve(source_func->parameters.size());
@@ -103,8 +102,7 @@ const string &InlineContentInjector::apply(Stage &stage, FunctionDeclaration &ta
 		SetForScope<Pass> set_pass(pass, RENAME);
 		var->visit(*this);
 
-		staging_block.body.push_back(0);
-		staging_block.body.back() = var;
+		staging_block.body.push_back_nocopy(var);
 		params.push_back(var);
 	}
 
@@ -118,8 +116,7 @@ const string &InlineContentInjector::apply(Stage &stage, FunctionDeclaration &ta
 		SetForScope<Pass> set_pass(pass, RENAME);
 		r_inlined_statement->visit(*this);
 
-		staging_block.body.push_back(0);
-		staging_block.body.back() = r_inlined_statement;
+		staging_block.body.push_back_nocopy(r_inlined_statement);
 	}
 
 	/* Now collect names from the staging block.  Local variables that would
@@ -133,7 +130,6 @@ const string &InlineContentInjector::apply(Stage &stage, FunctionDeclaration &ta
 	global identifiers used by the source function. */
 	pass = RENAME;
 	staging_block.parent = source_func->body.parent;
-	remap_prefix = target_func.name;
 	target_func.visit(*this);
 
 	// Put the argument expressions in place after all renaming has been done.
@@ -193,7 +189,7 @@ void InlineContentInjector::visit(VariableDeclaration &var)
 		staging_block.variables[var.name] = &var;
 		if(referenced_names.count(var.name))
 		{
-			string mapped_name = get_unused_variable_name(staging_block, var.name, remap_prefix);
+			string mapped_name = get_unused_variable_name(staging_block, var.name);
 			if(mapped_name!=var.name)
 			{
 				staging_block.variables[mapped_name] = &var;
@@ -217,7 +213,7 @@ void InlineContentInjector::visit(Return &ret)
 	if(pass==INLINE && ret.expression)
 	{
 		// Create a new variable to hold the return value of the inlined function.
-		r_result_name = get_unused_variable_name(staging_block, "_return", source_func->name);
+		r_result_name = get_unused_variable_name(staging_block, "_return");
 		RefPtr<VariableDeclaration> var = new VariableDeclaration;
 		var->source = ret.source;
 		var->line = ret.line;
@@ -285,7 +281,7 @@ void FunctionInliner::visit(FunctionCall &call)
 
 		// This will later get removed by UnusedVariableRemover.
 		if(result_name.empty())
-			result_name = "msp_unused_from_inline";
+			result_name = "_msp_unused_from_inline";
 
 		RefPtr<VariableReference> ref = new VariableReference;
 		ref->name = result_name;
@@ -1120,7 +1116,7 @@ void UnusedVariableRemover::visit(VariableDeclaration &var)
 	/* Mark variables as output if they're used by the next stage or the
 	graphics API. */
 	if(interface_block)
-		var_info.output = (interface_block->interface=="out" && (interface_block->linked_block || !interface_block->name.compare(0, 3, "gl_")));
+		var_info.output = (interface_block->interface=="out" && (interface_block->linked_block || !interface_block->block_name.compare(0, 3, "gl_")));
 	else
 		var_info.output = (var.interface=="out" && (stage->type==Stage::FRAGMENT || var.linked_declaration || !var.name.compare(0, 3, "gl_")));
 
@@ -1142,7 +1138,7 @@ void UnusedVariableRemover::visit(InterfaceBlock &iface)
 	else
 	{
 		VariableInfo &var_info = variables[&iface];
-		var_info.output = (iface.interface=="out" && (iface.linked_block || !iface.name.compare(0, 3, "gl_")));
+		var_info.output = (iface.interface=="out" && (iface.linked_block || !iface.block_name.compare(0, 3, "gl_")));
 	}
 }