[llvm-commits] Fix for MachineSink creating zombie defines

Jan Sjodin jan_sjodin at yahoo.com
Sat Jul 30 09:05:47 PDT 2011


>>  I was unsure if this is desirable for the generic function. Is that the 

> better place for this?
> 
> I can't imagine any user of SplitCriticalEdge would not want to
> preserve live-ins... and it looks like the other current caller,
> MachineLICM, suffers from the same issue.
> 


I attached a modified patch.

In MacineBasicBlock::SplitCriticalEdge it handles the case of LiveVariables,
Why is there both LiveVariables and livins?

- Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0051_machinesink.patch
Type: application/octet-stream
Size: 4801 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110730/a8f800b4/attachment.obj>


More information about the llvm-commits mailing list