> +++ llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp     Sun Jan 28  
> 07:31:35 2007
> @@ -2168,6 +2168,7 @@
>        const char *FnName = 0;
>        if (Node->getOpcode() == ISD::MEMSET) {
>          Entry.Node = Tmp2; Entry.isSigned = false; Entry.Ty =  
> IntPtrTy;
> +     Entry.isInReg = false;

Please remove the tabs here and elsewhere.

>    // Add one result value for each formal argument.
>    std::vector<MVT::ValueType> RetVals;
> +  unsigned j = 0;
>    for (Function::arg_iterator I = F.arg_begin(), E = F.arg_end();  
> I != E; ++I) {
>      MVT::ValueType VT = getValueType(I->getType());
> +    bool isInReg = FTy->paramHasAttr(++j,  
> FunctionType::InRegAttribute);

Please increment j in the for loop, e.g. "++I, ++j" to make it more  
clear how j is evolving.

Overall, this is a great patch, nice work!

-Chris


_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to