On Feb 7, 2008, at 8:28 AM, Dan Gohman wrote:
>   if (Flags & ISD::ParamFlags::ByVal) {
>     return CreateCopyOfByValArgument(Arg, PtrOff, Chain, Flags, DAG);
>   }
> -  return DAG.getStore(Chain, Arg, PtrOff, NULL, 0);
> +  return DAG.getStore(Chain, Arg, PtrOff,
> +                      &PseudoSourceValue::getStack(), LocMemOffset);
> }

Hey Dan,

Why not have getStack() return a Value* instead of Value&?  That would  
eliminate the '&'s and would be more consistent with other APIS,

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

Reply via email to