maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[llvm-exegesis] Rename range based for loop variable in a unit test so its different than the container being iterated over. NFC 

It seems like gcc 5.5 wants to iterate over the new variable instead
of the container that lives outside the loop. But of course this
new container is empty.

Plus using a different variable names makes the code more readable.
This commit is contained in:
Craig Topper 2020-02-14 13:41:50 -08:00
parent 8dc659c131
commit 1fe6e6fd85
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/tools/llvm-exegesis/lib/SnippetGenerator.h
View File

@ -165,8 +165,8 @@ class CombinationGenerator {
// Initialize the per-variable state to refer to the possible choices for
// that variable.
VariablesState.reserve(VariablesChoices.size());
for (ArrayRef<choice_type> VariablesChoices : VariablesChoices)
VariablesState.emplace_back(VariablesChoices);
for (ArrayRef<choice_type> VC : VariablesChoices)
VariablesState.emplace_back(VC);
// Temporary buffer to store each combination before performing Callback.
SmallVector<choice_type, variable_smallsize> CurrentCombination;