[LLVMdev] Question about GVN

Tue May 4 10:08:40 PDT 2010

On May 4, 2010, at 9:41 AM, Jakub Staszak wrote:
> 
> Because it iterates over predecessors it may happen that not the first one has IndirectBrInst terminator.
> However some "toSplit" edges can be already added. Is this OK?
> Maybe I just misunderstood something?

It might result in doing some unnecessary edge splitting, but it's not dangerous in the "breaking the program" sense.  Presumably SimplifyCFG will fold these blocks away if they're empty.

--Owen