diff options
author | Sergey Andreenko <seandree@microsoft.com> | 2017-09-21 21:30:15 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-21 21:30:15 -0700 |
commit | 43cf34fe74b435427ffb62f54fc730b4b46c4c22 (patch) | |
tree | 804ee54b70eb40841e29a7f7e31eea3dd92745dd /src/jit/lsraarm.cpp | |
parent | 8c33e99150cfe9c173f524acff4ea7be320ce08c (diff) | |
download | coreclr-43cf34fe74b435427ffb62f54fc730b4b46c4c22.tar.gz coreclr-43cf34fe74b435427ffb62f54fc730b4b46c4c22.tar.bz2 coreclr-43cf34fe74b435427ffb62f54fc730b4b46c4c22.zip |
fix checkLclVarSemantics (#14053)
* extract CheckLclVarSemantics from CheckLIR.
* add a test that shows the silent bad execution.
* fix the checker.
* add the test to the exclude list.
* rename consumed to used
Diffstat (limited to 'src/jit/lsraarm.cpp')
-rw-r--r-- | src/jit/lsraarm.cpp | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/src/jit/lsraarm.cpp b/src/jit/lsraarm.cpp index 00d466e923..a0ec93a524 100644 --- a/src/jit/lsraarm.cpp +++ b/src/jit/lsraarm.cpp @@ -441,19 +441,11 @@ void LinearScan::TreeNodeInfoInit(GenTree* tree) break; case GT_LONG: - if (tree->IsUnusedValue()) - { - // An unused GT_LONG node needs to consume its sources. - info->srcCount = 2; - info->dstCount = 0; - } - else - { - // Passthrough. Should have been marked contained. - info->srcCount = 0; - assert(info->dstCount == 0); - } + assert(tree->IsUnusedValue()); // Contained nodes are already processed, only unused GT_LONG can reach here. + // An unused GT_LONG node needs to consume its sources. + info->srcCount = 2; + info->dstCount = 0; break; case GT_CNS_DBL: |