diff options
| author | Evan Cheng <evan.cheng@apple.com> | 2009-04-21 22:46:52 +0000 |
|---|---|---|
| committer | Evan Cheng <evan.cheng@apple.com> | 2009-04-21 22:46:52 +0000 |
| commit | 047e9d1fa34f28a3db20f841eec1d194abf2a1da (patch) | |
| tree | 4f04dd784d95779528ce075c994f65335631067c /lib/CodeGen/RegAllocLinearScan.cpp | |
| parent | 101a267d8a32e6aef0bbe6fe1b5ff540828a00a4 (diff) | |
| download | external_llvm-047e9d1fa34f28a3db20f841eec1d194abf2a1da.tar.gz external_llvm-047e9d1fa34f28a3db20f841eec1d194abf2a1da.tar.bz2 external_llvm-047e9d1fa34f28a3db20f841eec1d194abf2a1da.zip | |
It has finally happened. Spiller is now using live interval info.
This fixes a very subtle bug. vr defined by an implicit_def is allowed overlap with any register since it doesn't actually modify anything. However, if it's used as a two-address use, its live range can be extended and it can be spilled. The spiller must take care not to emit a reload for the vn number that's defined by the implicit_def. This is both a correctness and performance issue.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@69743 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/RegAllocLinearScan.cpp')
| -rw-r--r-- | lib/CodeGen/RegAllocLinearScan.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/CodeGen/RegAllocLinearScan.cpp b/lib/CodeGen/RegAllocLinearScan.cpp index c738315484..2e65b3f937 100644 --- a/lib/CodeGen/RegAllocLinearScan.cpp +++ b/lib/CodeGen/RegAllocLinearScan.cpp @@ -345,7 +345,7 @@ bool RALinScan::runOnMachineFunction(MachineFunction &fn) { linearScan(); // Rewrite spill code and update the PhysRegsUsed set. - spiller_->runOnMachineFunction(*mf_, *vrm_); + spiller_->runOnMachineFunction(*mf_, *vrm_, li_); assert(unhandled_.empty() && "Unhandled live intervals remain!"); fixed_.clear(); |