Hi Chris,

On Feb 12, 2008, at 11:39 PM, Chris Lattner wrote:
>
> URL: http://llvm.org/viewvc/llvm-project?rev=47054&view=rev
> Log:
> In SDISel, for targets that support FORMAL_ARGUMENTS nodes, lower this
> node as soon as we create it in SDISel.  Previously we would lower  
> it in
> legalize.  The problem with this is that it only exposes the argument
> loads implied by FORMAL_ARGUMENTs after legalize, so that only dag  
> combine 2
> can hack on them.  This causes us to miss some optimizations because
> datatype expansion also happens here.

Will this become redundant once LegalizeTypes is finished and DAGCombine
can be run between it an legalizing operations?

Dan

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

Reply via email to