[llvm-commits] [llvm] r92873 - in /llvm/trunk: include/llvm/Target/TargetInstrInfo.h lib/CodeGen/TailDuplication.cpp lib/CodeGen/TargetInstrInfoImpl.cpp lib/Target/ARM/ARMBaseInstrInfo.cpp lib/Target/ARM/ARMBaseInstrInfo.h test/CodeGen/Thumb2/2010-01-06-TailDuplicateLabels.ll

[Dan Gohman]

On Jan 6, 2010, at 3:47 PM, Jakob Stoklund Olesen wrote:

> Author: stoklund
> Date: Wed Jan  6 17:47:07 2010
> New Revision: 92873
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=92873&view=rev
> Log:
> Add Target hook to duplicate machine instructions.
> 
> Some instructions refer to unique labels, and so cannot be trivially cloned
> with CloneMachineInstr.

It looks like this still requires that TargetInstrDesc::isNotDuplicable
be unset, right? Can you mention that it a comment?

Also, can you add a comment to MachineFunction::CloneMachineInstr
that mentions the new duplicate function?

Thanks,

Dan