[llvm-commits] [Patch] Add in expansions for various condition codes

[Villmow, Micah]

R165655.

> -----Original Message-----
> From: Tom Stellard [mailto:tom at stellard.net]
> Sent: Tuesday, October 09, 2012 7:32 AM
> To: Villmow, Micah
> Cc: Rafael Espíndola; llvm-commits at cs.uiuc.edu
> Subject: Re: [llvm-commits] [Patch] Add in expansions for various
> condition codes
> 
> On Mon, Oct 08, 2012 at 03:31:47PM +0000, Villmow, Micah wrote:
> > This is needed for the AMDIL backend. When I start getting to
> developing more on that(after TargetData/DataLayout work is done), then
> there will be a backend that can verify that these are correct with lit
> tests.
> >
> > Micah
> >
> 
> The R600 backend will use this as well.  I'll add some test cases to
> the R600 branch once this is merged.
> 
> -Tom
> 
> > > -----Original Message-----
> > > From: Rafael Espíndola [mailto:rafael.espindola at gmail.com]
> > > Sent: Saturday, October 06, 2012 7:26 PM
> > > To: Villmow, Micah
> > > Cc: Hal Finkel; Evan Cheng; llvm-commits at cs.uiuc.edu
> > > Subject: Re: [llvm-commits] [Patch] Add in expansions for various
> > > condition codes
> > >
> > > On 4 October 2012 14:09, Villmow, Micah <Micah.Villmow at amd.com>
> wrote:
> > > > Ping!
> > >
> > > +      // Otherwise, the pattern is (LHS CC1 LHS) Opc (RHS CC2
> RHS2)
> > >
> > > RHS2?
> > >
> > > The patch looks OK. Do you need it for some off tree backend or is
> > > this something we can test with an existing backend. If you can add
> a
> > > test, please do.
> > >
> > > Not sure if this would help MBlaze, the patters seem to be to
> convert
> > > from setcc to selectcc, so yes, it can be a separate patch.
> > >
> > > Cheers,
> > > Rafael
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits