[llvm-commits] MemIntrinsicNode patch

Mon Ping Wang wangmp at apple.com
Fri Oct 17 11:28:33 PDT 2008


Hi,

On Oct 16, 2008, at 10:59 PM, Evan Cheng wrote:

> +/// Memory intrinsic node
> +// This node is used for target intrinsic that touches memory and
> need an
> +// associated memory operan
>
> // -> /// :-)
>
- Fixed
> +  /// getMemIntrinsicNode - Gets a MemIntrinsicNode
> +SDValue getMemIntrinsicNode(unsigned Opcode,
> +                            const MVT *VTs, unsigned NumVTs,
> +                            const SDValue *Ops, unsigned NumOps,
> +                            MVT MemVT, const Value *srcValue, int
> SVOff,
> +                            unsigned Align = 0, bool Vol = false,
> +                            bool ReadMem = true, bool WriteMem =  
> true);
>
> Extra spaces before ///? Also please end a sentence with a period (or
> Chris' head will explode. :-)
>

-Fixed as well as the indentation problem

Thanks,
   -- Mon Ping





More information about the llvm-commits mailing list