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 | 1a99a5f501f26efa059a8c716217feaed11d9a43 (patch) | |
| tree | 61e36aacdaf0ff9416b89c46104e857d37f163a5 /llvm/lib/CodeGen/RegAllocPBQP.cpp | |
| parent | 41b869477718ab70a696fbb6911b6020fd53f11b (diff) | |
| download | bcm5719-llvm-1a99a5f501f26efa059a8c716217feaed11d9a43.tar.gz bcm5719-llvm-1a99a5f501f26efa059a8c716217feaed11d9a43.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.
llvm-svn: 69743
Diffstat (limited to 'llvm/lib/CodeGen/RegAllocPBQP.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/RegAllocPBQP.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/RegAllocPBQP.cpp b/llvm/lib/CodeGen/RegAllocPBQP.cpp index adab2b01d80..748fae48639 100644 --- a/llvm/lib/CodeGen/RegAllocPBQP.cpp +++ b/llvm/lib/CodeGen/RegAllocPBQP.cpp @@ -854,7 +854,7 @@ bool PBQPRegAlloc::runOnMachineFunction(MachineFunction &MF) { // Run spiller std::auto_ptr<Spiller> spiller(createSpiller()); - spiller->runOnMachineFunction(*mf, *vrm); + spiller->runOnMachineFunction(*mf, *vrm, lis); return true; } |
