[llvm-commits] [Review request] Moving code from PHIElimination to MachineBasicBlock

[Cameron Zwarich]

On Dec 5, 2010, at 10:38 AM, Chris Lattner wrote:

> On Dec 4, 2010, at 4:16 PM, Cameron Zwarich wrote:
> 
>> MachineBasicBlock should be responsible for the placement of phis in predecessor blocks, sparing PHIElimination the need to know the details. This will also let StrongPHIElimination share the code.
> 
> I don't have a strong opinion on this, but this function does seem pretty phi-elimination specific, and it seems strange for MBB.cpp to have to pull in MachineRegisterInfo.h.  If your goal is to share this between phi elimination implementation, it would probably be better in a PHIEliminationUtils.cpp file or something?

Here's an updated patch. Does it look okay?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: copy-insert-point.patch
Type: application/octet-stream
Size: 6709 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101205/ee64a5d9/attachment.obj>
-------------- next part --------------


Cameron