[PATCH] R600: Make sign_extend_inreg legal.

[Tom Stellard]

On Mon, Apr 21, 2014 at 09:54:04AM -0700, Matt Arsenault wrote:
> 
> On Apr 21, 2014, at 6:40 AM, Tom Stellard <tom at stellard.net> wrote:
> 
> > On Thu, Apr 17, 2014 at 10:37:12PM -0700, Matt Arsenault wrote:
> >> 
> >> On Apr 2, 2014, at 9:59 AM, Tom Stellard <tom at stellard.net> wrote:
> >> 
> >>> I want to move away from using patterns with INSERT_SUBREG, because
> >>> it just adds another case we need to handle in the SIFixSGPRCopies pass.
> >>> Can you move this pattern into AMDILISelDAGToDAG.cpp and use REG_SEQUENCE
> >>> instead of INSERT_SUBREG + IMPLICIT_DEF.
> >> 
> >> Now that it handles INSERT_SUBREG anyway, should I go back to using the patterns?
> >> 
> > 
> > What are the advantages of using the patterns?
> 
> Less code

Using the patterns is fine with me (sorry I went back and forth on this).

-Tom