[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

[Jakob Stoklund Olesen]

On Jan 6, 2010, at 4:40 PM, Dan Gohman wrote:

> 
> 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?

r92883, thanks!