diff options
author | Andy Ayers <andya@microsoft.com> | 2018-07-18 14:35:08 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-07-18 14:35:08 -0700 |
commit | b2842bbef5162383f7bf67de2976f2c21dbfdb1d (patch) | |
tree | 3b03b3f0a166eb8518804b99913ef0311322fa2f /src/jit/lowerxarch.cpp | |
parent | 4037baf5675ec275c624395ef2b0337cf58836cd (diff) | |
download | coreclr-b2842bbef5162383f7bf67de2976f2c21dbfdb1d.tar.gz coreclr-b2842bbef5162383f7bf67de2976f2c21dbfdb1d.tar.bz2 coreclr-b2842bbef5162383f7bf67de2976f2c21dbfdb1d.zip |
JIT: force all local var ref counts to be accessed via API (#18979)
This is a preparatory change for auditing and controlling how local
variable ref counts are observed and manipulated.
See #18969 for context.
No diffs seen locally. No TP impact expected.
There is a small chance we may see some asserts in broader testing
as there were places in original code where local ref counts were
incremented without checking for possible overflows. The new APIs
will assert for overflow cases.
Diffstat (limited to 'src/jit/lowerxarch.cpp')
-rw-r--r-- | src/jit/lowerxarch.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/jit/lowerxarch.cpp b/src/jit/lowerxarch.cpp index 938ddb5862..327eb2b8cc 100644 --- a/src/jit/lowerxarch.cpp +++ b/src/jit/lowerxarch.cpp @@ -1338,7 +1338,7 @@ GenTree* Lowering::PreferredRegOptionalOperand(GenTree* tree) // weight as reg optional. // If either is not tracked, it may be that it was introduced after liveness // was run, in which case we will always prefer op1 (should we use raw refcnt??). - if (v1->lvTracked && v2->lvTracked && (v1->lvRefCntWtd >= v2->lvRefCntWtd)) + if (v1->lvTracked && v2->lvTracked && (v1->lvRefCntWtd() >= v2->lvRefCntWtd())) { preferredOp = op2; } |