diff options
| author | 2014-09-29 12:00:40 +0100 | |
|---|---|---|
| committer | 2014-09-30 13:50:38 +0100 | |
| commit | 8ddb00ca935733f5d3b07816e5bb33d6cabe6ec4 (patch) | |
| tree | 9bca67b136523eb31aab736988143295ece97b56 /compiler/optimizing/ssa_liveness_analysis.cc | |
| parent | cc6b59ee25d7b9782cc971687715d664a97b05bd (diff) | |
Improve detection of lifetime holes.
The check concluding that the next use was in a successor
was too conservative: two blocks following each other
in terms of liveness are not necessarily predecessor/sucessor.
Change-Id: Ideec98046c812aa5fb63781141b5fde24c706d6d
Diffstat (limited to 'compiler/optimizing/ssa_liveness_analysis.cc')
| -rw-r--r-- | compiler/optimizing/ssa_liveness_analysis.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/compiler/optimizing/ssa_liveness_analysis.cc b/compiler/optimizing/ssa_liveness_analysis.cc index 680cc0a033..cd13d81a36 100644 --- a/compiler/optimizing/ssa_liveness_analysis.cc +++ b/compiler/optimizing/ssa_liveness_analysis.cc @@ -189,6 +189,7 @@ void SsaLivenessAnalysis::ComputeLiveRanges() { } // Add a range that covers this block to all instructions live_in because of successors. + // Instructions defined in this block will have their start of the range adjusted. for (uint32_t idx : live_in->Indexes()) { HInstruction* current = instructions_from_ssa_index_.Get(idx); current->GetLiveInterval()->AddRange(block->GetLifetimeStart(), block->GetLifetimeEnd()); |