diff options
author | Sergey Andreenko <seandree@microsoft.com> | 2019-03-27 10:59:04 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-03-27 10:59:04 -0700 |
commit | cbeadacafba718557e47fc166d7fd688b01515d7 (patch) | |
tree | 453639f037b33ca1df630bf30b1343f3d3ecbc22 /src/jit/valuenum.cpp | |
parent | 6cb120cc74ca61fc314e3cc43007778d2aafd041 (diff) | |
download | coreclr-cbeadacafba718557e47fc166d7fd688b01515d7.tar.gz coreclr-cbeadacafba718557e47fc166d7fd688b01515d7.tar.bz2 coreclr-cbeadacafba718557e47fc166d7fd688b01515d7.zip |
Revert "Delete unused variables in jit. Part 2. (#23481)" (#23488)
This reverts commit 6cb120cc74ca61fc314e3cc43007778d2aafd041.
Diffstat (limited to 'src/jit/valuenum.cpp')
-rw-r--r-- | src/jit/valuenum.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/jit/valuenum.cpp b/src/jit/valuenum.cpp index 4280d872b0..fb31d42667 100644 --- a/src/jit/valuenum.cpp +++ b/src/jit/valuenum.cpp @@ -5923,6 +5923,8 @@ void Compiler::fgValueNumberBlock(BasicBlock* blk) compCurStmtNum = blk->bbStmtNum - 1; // Set compCurStmtNum #endif + unsigned outerLoopNum = BasicBlock::NOT_IN_LOOP; + // First: visit phi's. If "newVNForPhis", give them new VN's. If not, // first check to see if all phi args have the same value. GenTree* firstNonPhi = blk->FirstNonPhiDef(); @@ -7847,6 +7849,7 @@ void Compiler::fgValueNumberTree(GenTree* tree) // can recognize redundant loads with no stores between them. GenTree* addr = tree->AsIndir()->Addr(); GenTreeLclVarCommon* lclVarTree = nullptr; + FieldSeqNode* fldSeq1 = nullptr; FieldSeqNode* fldSeq2 = nullptr; GenTree* obj = nullptr; GenTree* staticOffset = nullptr; @@ -7888,6 +7891,9 @@ void Compiler::fgValueNumberTree(GenTree* tree) ValueNum inxVN = ValueNumStore::NoVN; FieldSeqNode* fldSeq = nullptr; + // GenTree* addr = tree->gtOp.gtOp1; + ValueNum addrVN = addrNvnp.GetLiberal(); + // Try to parse it. GenTree* arr = nullptr; addr->ParseArrayAddress(this, &arrInfo, &arr, &inxVN, &fldSeq); @@ -9743,8 +9749,7 @@ void Compiler::JitTestCheckVN() } // The mapping(s) must be one-to-one: if the label has a mapping, then the ssaNm must, as well. ssize_t num2; - bool found = vnToLabel->Lookup(vn, &num2); - assert(found); + bool b = vnToLabel->Lookup(vn, &num2); // And the mappings must be the same. if (tlAndN.m_num != num2) { |