LocalStackSlotAllocation improvements
authorHal Finkel <hfinkel@anl.gov>
Tue, 30 Apr 2013 20:04:37 +0000 (20:04 +0000)
committerHal Finkel <hfinkel@anl.gov>
Tue, 30 Apr 2013 20:04:37 +0000 (20:04 +0000)
commitdb31bd31d62b5b85dddd5fbecae1a04a02201adc
treecbb1778a4622acb5be5f9521d8948cc03d5d3dfd
parent34f39841d3e7929c5722cee3c27aefbca482d81a
LocalStackSlotAllocation improvements

First, taking advantage of the fact that the virtual base registers are allocated in order of the local frame offsets, remove the quadratic register-searching behavior. Because of the ordering, we only need to check the last virtual base register created.

Second, store the frame index in the FrameRef structure, and get the frame index and the local offset from this structure at the top of the loop iteration. This allows us to de-nest the loops in insertFrameReferenceRegisters (and I think makes the code cleaner). I also moved the needsFrameBaseReg check into the first loop over instructions so that we don't bother pushing FrameRefs for instructions that don't want a virtual base register anyway.

Lastly, and this is the only functionality change, avoid the creation of single-use virtual base registers. These are currently not useful because, in general, they end up replacing what would be one r+r instruction with an add and a r+i instruction. Committing this removes the XFAIL in CodeGen/PowerPC/2007-09-07-LoadStoreIdxForms.ll

Jim has okayed this off-list.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@180799 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/LocalStackSlotAllocation.cpp
test/CodeGen/PowerPC/2007-09-07-LoadStoreIdxForms.ll
test/CodeGen/Thumb/large-stack.ll