[llvm-commits] [llvm] r93220 - /llvm/trunk/include/llvm/CodeGen/MachineOperand.h

[Chris Lattner]

On Jan 11, 2010, at 6:01 PM, Dale Johannesen wrote:

> Author: johannes
> Date: Mon Jan 11 20:01:53 2010
> New Revision: 93220
>
> URL: http://llvm.org/viewvc/llvm-project?rev=93220&view=rev
> Log:
> Add MO_Metadata as an operand kind.  Not used yet.

Hey Dale,

All of this work looks great, one minor thing:

> +  static MachineOperand CreateMetadata(MDNode *Meta,
> +                                       unsigned char TargetFlags =  
> 0) {
> +    MachineOperand Op(MachineOperand::MO_Metadata);
> +    Op.Contents.MD = Meta;
> +    Op.setTargetFlags(TargetFlags);
> +    return Op;
> +  }

You don't need the TargetFlags argument here.

-Chris