X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fglsl%2Fresolve.cpp;h=3201ea1bb2a70665507dd64469916fb0b09daa5c;hb=631872e13c2670f7dece44512ed55af50b77f399;hp=02f224d66b4453d2469a2638e0ab0453d2cd6c4a;hpb=03d9003072e85c934f5624329fb4a34db8763db2;p=libs%2Fgl.git

diff --git a/source/glsl/resolve.cpp b/source/glsl/resolve.cpp
index 02f224d6..3201ea1b 100644
--- a/source/glsl/resolve.cpp
+++ b/source/glsl/resolve.cpp
@@ -225,29 +225,43 @@ void VariableResolver::visit(VariableReference &var)
 	if(!declaration)
 	{
 		const map<string, InterfaceBlock *> &blocks = stage->interface_blocks;
-		map<string, InterfaceBlock *>::const_iterator i = blocks.find("_"+var.name);
+		map<string, InterfaceBlock *>::const_iterator i = blocks.find(var.name);
+		if(i==blocks.end())
+		{
+			// Look for the variable in anonymous interface blocks.
+			for(i=blocks.begin(); i!=blocks.end(); ++i)
+				if(i->second->instance_name.empty() && i->second->struct_declaration)
+					if(i->second->struct_declaration->members.variables.count(var.name))
+						break;
+		}
+
 		if(i!=blocks.end())
 		{
-			/* The name refers to an interface block with an instance name rather
-			than a variable.  Prepare a new syntax tree node accordingly. */
+			/* The name refers to either an interface block with an instance name
+			or a variable declared inside an anonymous interface block.  Prepare
+			new syntax tree nodes accordingly. */
 			InterfaceBlockReference *iface_ref = new InterfaceBlockReference;
 			iface_ref->source = var.source;
 			iface_ref->line = var.line;
-			iface_ref->name = var.name;
 			iface_ref->declaration = i->second;
-			r_replacement_expr = iface_ref;
-		}
-		else
-		{
-			// Look for the variable in anonymous interface blocks.
-			for(i=blocks.begin(); (!declaration && i!=blocks.end()); ++i)
-				if(i->second->instance_name.empty() && i->second->struct_declaration)
-				{
-					const map<string, VariableDeclaration *> &iface_vars = i->second->struct_declaration->members.variables;
-					map<string, VariableDeclaration *>::const_iterator j = iface_vars.find(var.name);
-					if(j!=iface_vars.end())
-						declaration = j->second;
-				}
+
+			if(i->second->instance_name.empty())
+			{
+				iface_ref->name = format("%s %s", i->second->interface, i->second->block_name);
+
+				MemberAccess *memacc = new MemberAccess;
+				memacc->source = var.source;
+				memacc->line = var.line;
+				memacc->left = iface_ref;
+				memacc->member = var.name;
+
+				r_replacement_expr = memacc;
+			}
+			else
+			{
+				iface_ref->name = var.name;
+				r_replacement_expr = iface_ref;
+			}
 		}
 	}
 
@@ -259,7 +273,7 @@ void VariableResolver::visit(VariableReference &var)
 
 void VariableResolver::visit(InterfaceBlockReference &iface)
 {
-	map<string, InterfaceBlock *>::iterator i = stage->interface_blocks.find("_"+iface.name);
+	map<string, InterfaceBlock *>::iterator i = stage->interface_blocks.find(iface.name);
 	InterfaceBlock *declaration = (i!=stage->interface_blocks.end() ? i->second : 0);
 	r_any_resolved |= (declaration!=iface.declaration);
 	iface.declaration = declaration;
@@ -267,13 +281,6 @@ void VariableResolver::visit(InterfaceBlockReference &iface)
 	check_assignment_target(iface.declaration);
 }
 
-void VariableResolver::add_to_chain(Assignment::Target::ChainType type, unsigned index)
-{
-	if(r_assignment_target.chain_len<7)
-		r_assignment_target.chain[r_assignment_target.chain_len] = type | min<unsigned>(index, 0x3F);
-	++r_assignment_target.chain_len;
-}
-
 void VariableResolver::visit(MemberAccess &memacc)
 {
 	TraversingVisitor::visit(memacc);
@@ -286,11 +293,12 @@ void VariableResolver::visit(MemberAccess &memacc)
 		if(i!=strct->members.variables.end())
 		{
 			declaration = i->second;
+			index = 0;
 			for(NodeList<Statement>::const_iterator j=strct->members.body.begin(); (j!=strct->members.body.end() && j->get()!=i->second); ++j)
 				++index;
 
 			if(record_target)
-				add_to_chain(Assignment::Target::MEMBER, index);
+				add_to_chain(r_assignment_target, Assignment::Target::MEMBER, index);
 		}
 	}
 	else if(BasicTypeDeclaration *basic = dynamic_cast<BasicTypeDeclaration *>(memacc.left->type))
@@ -335,7 +343,7 @@ void VariableResolver::visit(Swizzle &swizzle)
 		unsigned mask = 0;
 		for(unsigned i=0; i<swizzle.count; ++i)
 			mask |= 1<<swizzle.components[i];
-		add_to_chain(Assignment::Target::SWIZZLE, mask);
+		add_to_chain(r_assignment_target, Assignment::Target::SWIZZLE, mask);
 	}
 }
 
@@ -357,7 +365,7 @@ void VariableResolver::visit(BinaryExpression &binary)
 			if(Literal *literal_subscript = dynamic_cast<Literal *>(binary.right.get()))
 				if(literal_subscript->value.check_type<int>())
 					index = literal_subscript->value.value<int>();
-			add_to_chain(Assignment::Target::ARRAY, index);
+			add_to_chain(r_assignment_target, Assignment::Target::ARRAY, index);
 		}
 	}
 	else
@@ -430,9 +438,9 @@ void VariableResolver::visit(InterfaceBlock &iface)
 {
 	/* Block names can be reused in different interfaces.  Prefix the name with
 	the first character of the interface to avoid conflicts. */
-	stage->interface_blocks.insert(make_pair(iface.interface+iface.block_name, &iface));
+	stage->interface_blocks.insert(make_pair(format("%s %s", iface.interface, iface.block_name), &iface));
 	if(!iface.instance_name.empty())
-		stage->interface_blocks.insert(make_pair("_"+iface.instance_name, &iface));
+		stage->interface_blocks.insert(make_pair(iface.instance_name, &iface));
 
 	TraversingVisitor::visit(iface);
 }