commit 1c89da46ff48a20ffc60ccdfad5c1f8b44f123b5
author GregF <greg@LunarG.com> Wed Apr 18 11:55:10 2018 -0600
committer David Neto <dneto@google.com> Thu Apr 19 15:34:15 2018 -0400
tree ab78b893f4cf7801dbeafbcc604e7c561b4381e6
parent df7f00f60ed61f9bf88e51c9af2a9bd05b8ef930

Test/DependencyAnalysis: Fix uninitialized variables

test/opt/loop_optimizations/dependence_analysis.cpp

diff --git a/test/opt/loop_optimizations/dependence_analysis.cpp b/test/opt/loop_optimizations/dependence_analysis.cpp
index a9799ca..3241c89 100644
--- a/test/opt/loop_optimizations/dependence_analysis.cpp
+++ b/test/opt/loop_optimizations/dependence_analysis.cpp
@@ -1419,7 +1419,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 29)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1440,7 +1440,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 54)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1462,7 +1462,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 75)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1483,7 +1483,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 99)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1505,7 +1505,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 121)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1526,7 +1526,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 142)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1548,7 +1548,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 162)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;
@@ -1569,7 +1569,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store;
+    const ir::Instruction* store = nullptr;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 183)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
         store = &inst;

test/opt/loop_optimizations/dependence_analysis_helpers.cpp

diff --git a/test/opt/loop_optimizations/dependence_analysis_helpers.cpp b/test/opt/loop_optimizations/dependence_analysis_helpers.cpp
index 37120ee..f4c5e27 100644
--- a/test/opt/loop_optimizations/dependence_analysis_helpers.cpp
+++ b/test/opt/loop_optimizations/dependence_analysis_helpers.cpp
@@ -170,7 +170,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store[1];
+    const ir::Instruction* store[1] = {nullptr};
     int stores_found = 0;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 16)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {
@@ -197,7 +197,7 @@
     std::vector<const ir::Loop*> loops{loop};
     opt::LoopDependenceAnalysis analysis{context.get(), loops};
 
-    const ir::Instruction* store[1];
+    const ir::Instruction* store[1] = {nullptr};
     int stores_found = 0;
     for (const ir::Instruction& inst : *spvtest::GetBasicBlock(f, 47)) {
       if (inst.opcode() == SpvOp::SpvOpStore) {