From bb4c0b1af6787da57b95219599f7abf473f7d820 Mon Sep 17 00:00:00 2001 From: David Neto Date: Tue, 26 Jul 2016 14:59:40 -0400 Subject: [PATCH] Rename a variable so it's consistent with spec In the spec "continue block" is a block with a branch to the Continue Target. It's not the Continue Target. --- source/val/Function.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/source/val/Function.cpp b/source/val/Function.cpp index 8d9442a..73007c1 100644 --- a/source/val/Function.cpp +++ b/source/val/Function.cpp @@ -134,17 +134,18 @@ spv_result_t Function::RegisterLoopMerge(uint32_t merge_id, RegisterBlock(merge_id, false); RegisterBlock(continue_id, false); BasicBlock& merge_block = blocks_.at(merge_id); - BasicBlock& continue_block = blocks_.at(continue_id); + BasicBlock& continue_target_block = blocks_.at(continue_id); assert(current_block_ && "RegisterLoopMerge must be called when called within a block"); current_block_->set_type(kBlockTypeLoop); merge_block.set_type(kBlockTypeMerge); - continue_block.set_type(kBlockTypeContinue); + continue_target_block.set_type(kBlockTypeContinue); cfg_constructs_.emplace_back(ConstructType::kLoop, current_block_, &merge_block); Construct& loop_construct = cfg_constructs_.back(); - cfg_constructs_.emplace_back(ConstructType::kContinue, &continue_block); + cfg_constructs_.emplace_back(ConstructType::kContinue, + &continue_target_block); Construct& continue_construct = cfg_constructs_.back(); continue_construct.set_corresponding_constructs({&loop_construct}); loop_construct.set_corresponding_constructs({&continue_construct}); -- 2.7.4