diff options
author | Carol Eidt <carol.eidt@microsoft.com> | 2017-07-31 17:41:13 -0700 |
---|---|---|
committer | Carol Eidt <carol.eidt@microsoft.com> | 2017-08-16 11:37:16 -0700 |
commit | 0ad49b141f1d06c5e83ec58fa7f21c7d2f3714d3 (patch) | |
tree | 717327f4a7f0fabc9d1a7c83681504818d6d01f5 /src/jit/rationalize.cpp | |
parent | 89d8be12677a5d395d1ec7e62874a4476ea5d4d6 (diff) | |
download | coreclr-0ad49b141f1d06c5e83ec58fa7f21c7d2f3714d3.tar.gz coreclr-0ad49b141f1d06c5e83ec58fa7f21c7d2f3714d3.tar.bz2 coreclr-0ad49b141f1d06c5e83ec58fa7f21c7d2f3714d3.zip |
Move Containment Analysis to Lowering
- Ensure that the only changes to child nodes are the specification of srcCandidates
(i.e. the register requirements of the parent/consumer)
- Since this now occurs before redoing dataflow, we don't have complete information on
what lclVars are containable. To mitigate this:
- In `TreeNodeInfoInit()`, if a lclVar is marked `isRegOptional` but is now untracked,
change it to be contained.
- Mark decomposed/promoted fields of long params as `lvDoNotEnregister`
- Ensure that any new code added after `Lowering` is analyzed as necessary.
- When a value becomes unused, it is no longer contained. (This is needed now because liveness runs after containment analysis)
- Add dumping of nodes deleted as unused.
- Need to set byteable regs, as needed on contained RMW memory ops.
Diffstat (limited to 'src/jit/rationalize.cpp')
-rw-r--r-- | src/jit/rationalize.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/jit/rationalize.cpp b/src/jit/rationalize.cpp index 5244c7424d..257e02d491 100644 --- a/src/jit/rationalize.cpp +++ b/src/jit/rationalize.cpp @@ -516,6 +516,7 @@ void Rationalizer::RewriteAssignment(LIR::Use& use) location->gtType = TYP_BYREF; assignment->SetOper(GT_STOREIND); + assignment->AsStoreInd()->SetRMWStatusDefault(); // TODO: JIT dump } |