X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=blobdiff_plain;f=source%2Fglsl%2Fdebug.cpp;h=5b142019150441f3fd7291f03db30473e401631b;hp=e9eb18c819d00522e4877f49dd04f51b582d2352;hb=b7ecc29c204faede028556d1942b2d61d5cda9ee;hpb=1205396c4cfdce84cf6559557ee7cb268a8c361c diff --git a/source/glsl/debug.cpp b/source/glsl/debug.cpp index e9eb18c8..5b142019 100644 --- a/source/glsl/debug.cpp +++ b/source/glsl/debug.cpp @@ -8,27 +8,20 @@ namespace Msp { namespace GL { namespace SL { -const std::string &DumpTree::apply(Stage &stage) +string DumpTree::apply(Stage &stage) { formatted = format("Stage: %s\n", Stage::get_stage_name(stage.type)); - tree.push_back(BRANCH); + begin_sub(); append(format("Version: %d.%02d", stage.required_features.glsl_version.major, stage.required_features.glsl_version.minor)); - for(std::map::const_iterator i=stage.types.begin(); i!=stage.types.end(); ++i) - append(format("Type: %%%d %s", get_label(*i->second), i->first)); + for(const auto &kvp: stage.types) + append(format("Type: %%%d %s", get_label(*kvp.second), kvp.first)); - set seen_interfaces; - for(std::map::const_iterator i=stage.interface_blocks.begin(); i!=stage.interface_blocks.end(); ++i) - if(seen_interfaces.insert(i->second).second) - { - string text = format("Interface block: %%%d %s %s", get_label(*i->second), i->second->interface, i->second->block_name); - if(!i->second->instance_name.empty()) - text += format(" %s", i->second->instance_name); - append(text); - } + for(const auto &kvp: stage.interface_blocks) + append(format("Interface block: %%%d %s", get_label(*kvp.second), kvp.first)); - for(std::map::const_iterator i=stage.functions.begin(); i!=stage.functions.end(); ++i) - append(format("Function: %%%d %s", get_label(*i->second), i->first)); + for(const auto &kvp: stage.functions) + append(format("Function: %%%d %s", get_label(*kvp.second), kvp.first)); last_branch(); stage.content.visit(*this); @@ -38,7 +31,7 @@ const std::string &DumpTree::apply(Stage &stage) void DumpTree::append(const string &line) { StringCodec::Utf8::Encoder enc; - for(vector::const_iterator i=tree.begin(); i!=tree.end(); ) + for(auto i=tree.begin(); i!=tree.end(); ) { enc.encode_char(*i++, formatted); enc.encode_char((i==tree.end() ? REACH : EMPTY), formatted); @@ -55,15 +48,19 @@ void DumpTree::append(const Node &node, const string &line) void DumpTree::append_subtree(const vector &branches) { begin_sub(); - for(vector::const_iterator i=branches.begin(); i!=branches.end(); ) + for(auto i=branches.begin(); i!=branches.end(); ) { - vector::const_iterator j = increment(i, branches); + auto j = increment(i, branches); if(!j->text.empty()) { + append(j->text); if(j->node) - annotated_branch(j->text, *j->node); - else - append(j->text); + { + begin_sub(); + last_branch(); + j->node->visit(*this); + end_sub(); + } } else j->node->visit(*this); @@ -81,7 +78,8 @@ void DumpTree::append_subtree(Node &node) void DumpTree::begin_sub() { - tree.back() = (tree.back()==BRANCH_LAST ? EMPTY : STRAIGHT); + if(!tree.empty()) + tree.back() = (tree.back()==BRANCH_LAST ? EMPTY : STRAIGHT); tree.push_back(BRANCH); } @@ -93,19 +91,10 @@ void DumpTree::last_branch() void DumpTree::end_sub() { tree.pop_back(); - if(tree.back()==STRAIGHT) + if(!tree.empty() && tree.back()==STRAIGHT) tree.back() = BRANCH; } -void DumpTree::annotated_branch(const string &annotation, Node &node) -{ - append(annotation); - begin_sub(); - last_branch(); - node.visit(*this); - end_sub(); -} - unsigned DumpTree::get_label(const Node &node) { unsigned &label = node_labels[&node]; @@ -122,7 +111,7 @@ string DumpTree::format_type(TypeDeclaration *type) template typename T::const_iterator DumpTree::increment(typename T::const_iterator &iter, const T &container) { - typename T::const_iterator ret = iter++; + auto ret = iter++; if(iter==container.end()) last_branch(); return ret; @@ -133,23 +122,14 @@ void DumpTree::visit(Block &block) append(block, format("Block %s", (block.use_braces ? "{}" : "(inline)"))); begin_sub(); - for(std::map::const_iterator i=block.variables.begin(); i!=block.variables.end(); ++i) - append(format("Variable: %%%d %s %s", get_label(*i->second), i->second->type, i->first)); + for(const auto &kvp: block.variables) + append(format("Variable: %%%d %s %s", get_label(*kvp.second), kvp.second->type, kvp.first)); - bool labeled_body = !block.variables.empty(); - if(labeled_body) - { - last_branch(); - append("Body"); - begin_sub(); - } - for(NodeList::const_iterator i=block.body.begin(); i!=block.body.end(); ) + for(auto i=block.body.cbegin(); i!=block.body.cend(); ) { - NodeList::const_iterator j = increment(i, block.body); + auto j = increment(i, block.body); (*j)->visit(*this); } - if(labeled_body) - end_sub(); end_sub(); } @@ -182,7 +162,7 @@ void DumpTree::visit(MemberAccess &memacc) string text = "Member access:"; if(memacc.declaration) text += format(" %%%d", get_label(*memacc.declaration)); - text += format(" .%s -> %s", memacc.member, format_type(memacc.type)); + text += format(" .%s (%d) -> %s", memacc.member, memacc.index, format_type(memacc.type)); append(memacc, text); append_subtree(*memacc.left); } @@ -274,9 +254,9 @@ void DumpTree::visit(FunctionCall &call) append(call, head); begin_sub(); - for(NodeArray::const_iterator i=call.arguments.begin(); i!=call.arguments.end(); ) + for(auto i=call.arguments.cbegin(); i!=call.arguments.cend(); ) { - NodeArray::const_iterator j = increment(i, call.arguments); + auto j = increment(i, call.arguments); (*j)->visit(*this); } end_sub(); @@ -302,9 +282,9 @@ void DumpTree::visit(Layout &layout) { append(layout, "Layout"); begin_sub(); - for(vector::const_iterator i=layout.qualifiers.begin(); i!=layout.qualifiers.end(); ) + for(auto i=layout.qualifiers.cbegin(); i!=layout.qualifiers.cend(); ) { - vector::const_iterator j = increment(i, layout.qualifiers); + auto j = increment(i, layout.qualifiers); string qualifier = j->name; if(j->has_value) qualifier += format("=%d", j->value); @@ -335,7 +315,7 @@ void DumpTree::visit(BasicTypeDeclaration &type) void DumpTree::visit(ImageTypeDeclaration &type) { - static const char *dims[] = { "1D", "2D", "3D", "Cube" }; + static const char *const dims[] = { "1D", "2D", "3D", "Cube" }; append(type, format("%%%d typedef %s", get_label(type), type.name)); @@ -393,10 +373,7 @@ void DumpTree::visit(VariableDeclaration &var) void DumpTree::visit(InterfaceBlock &iface) { - string head; - if(!iface.instance_name.empty()) - head += format("%%%d ", get_label(iface)); - head += format("%s %s", iface.interface, iface.block_name); + string head = format("%%%d %s %s", get_label(iface), iface.interface, iface.block_name); if(!iface.instance_name.empty()) head += format(" %s", iface.instance_name); if(iface.array) @@ -409,11 +386,11 @@ void DumpTree::visit(InterfaceBlock &iface) vector branches; if(iface.type_declaration) - append(format("Type: %%%d %s", get_label(*iface.type_declaration), iface.type_declaration->name)); + branches.push_back(format("Type: %%%d %s", get_label(*iface.type_declaration), iface.type_declaration->name)); if(iface.layout) - branches.push_back(Branch("Layout", iface.layout.get())); + branches.push_back(iface.layout.get()); if(iface.members) - branches.push_back(Branch("Members", iface.members.get())); + branches.push_back(iface.members.get()); append_subtree(branches); } @@ -429,8 +406,8 @@ void DumpTree::visit(FunctionDeclaration &func) begin_sub(); if(func.return_type_declaration) append(format("Return type: %%%d %s", get_label(*func.return_type_declaration), func.return_type_declaration->name)); - for(NodeArray::const_iterator i=func.parameters.begin(); i!=func.parameters.end(); ++i) - (*i)->visit(*this); + for(const RefPtr &p: func.parameters) + p->visit(*this); last_branch(); if(func.definition==&func) func.body.visit(*this); @@ -445,9 +422,9 @@ void DumpTree::visit(Conditional &cond) vector branches; branches.push_back(cond.condition.get()); - branches.push_back(&cond.body); + branches.push_back(Branch("then", &cond.body)); if(!cond.else_body.body.empty()) - branches.push_back(&cond.else_body); + branches.push_back(Branch("else", &cond.else_body)); append_subtree(branches); } @@ -455,16 +432,15 @@ void DumpTree::visit(Iteration &iter) { append(iter, "for()"); - begin_sub(); + vector branches; if(iter.init_statement) - annotated_branch("Initialization", *iter.init_statement); + branches.push_back(Branch("Initialization", iter.init_statement.get())); if(iter.condition) - annotated_branch("Condition", *iter.condition); + branches.push_back(Branch("Condition", iter.condition.get())); if(iter.loop_expression) - annotated_branch("Loop", *iter.loop_expression); - last_branch(); - annotated_branch("Body", iter.body); - end_sub(); + branches.push_back(Branch("Loop", iter.loop_expression.get())); + branches.push_back(&iter.body); + append_subtree(branches); } void DumpTree::visit(Passthrough &pass)