[llvm-commits] [llvm] r170456 - in /llvm/trunk: include/llvm/CodeGen/MachineBasicBlock.h lib/CodeGen/MachineBasicBlock.cpp

[Jakob Stoklund Olesen]

On Jan 17, 2013, at 7:42 AM, Krzysztof Parzyszek <kparzysz at codeaurora.org> wrote:

> On 12/18/2012 2:59 PM, Jakob Stoklund Olesen wrote:
>> Author: stoklund
>> Date: Tue Dec 18 14:59:41 2012
>> New Revision: 170456
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=170456&view=rev
>> Log:
>> Tighten up the splice() API for bundled instructions.
>> 
>> Remove the instr_iterator versions of the splice() functions. It doesn't
>> seem useful to be able to splice sequences of instructions that don't
>> consist of full bundles.
> 
> Ok, I'm a bit late to see this, but...
> 
> How about this:
> 
> BUNDLE
> * Instr1
> * Instr2
> * Instr3
> 
> and now I want to take one instruction out and place it outside of the bundle?  Now I will need to update the neighbors, which needs to take care of the corner cases (bundle boundaries).  I don't see any convenient way of doing this in the current code (mid-January).

MI->removeFromBundle().

The whole point of the new bundle API is that you don't need to update the bundling flags manually.

/jakob