From: David Neto Date: Tue, 26 Jul 2016 18:59:40 +0000 (-0400) Subject: Rename a variable so it's consistent with spec X-Git-Tag: upstream/2018.6~1169 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bb4c0b1af6787da57b95219599f7abf473f7d820;p=platform%2Fupstream%2FSPIRV-Tools.git 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. --- 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});