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

Villmow, Micah Micah.Villmow at amd.com
Thu Sep 27 09:43:47 PDT 2012


That is a good question. Evan?  Can this patch go in and then remove the expansions in MBlaze in a separate patch?

> -----Original Message-----
> From: Hal Finkel [mailto:hfinkel at anl.gov]
> Sent: Thursday, September 20, 2012 2:56 PM
> To: Villmow, Micah
> Cc: Duncan Sands; llvm-commits at cs.uiuc.edu
> Subject: Re: [llvm-commits] [Patch] Add in expansions for various
> condition codes
> 
> On Thu, 20 Sep 2012 21:38:28 +0000
> "Villmow, Micah" <Micah.Villmow at amd.com> wrote:
> 
> > They probably could since they look to be following a similar pattern,
> > but I would prefer that the owner of that code comment on if they
> > could or not.
> 
> Is there a current maintainer?
> 
>  -Hal
> 
> >
> > > -----Original Message-----
> > > From: Hal Finkel [mailto:hfinkel at anl.gov]
> > > Sent: Thursday, September 20, 2012 2:30 PM
> > > To: Villmow, Micah
> > > Cc: Duncan Sands; llvm-commits at cs.uiuc.edu
> > > Subject: Re: [llvm-commits] [Patch] Add in expansions for various
> > > condition codes
> > >
> > > Micah,
> > >
> > > Thanks for working on this.
> > >
> > > Can these expansions be used to replace the patterns in
> > > MBlazeInstrFPU.td and MBlazeInstrInfo.td? If so, then we should do
> > > so, especially because a lot of people seem to use MBlaze as the
> > > "template" backend.
> > >
> > >  -Hal
> > >
> > > On Thu, 20 Sep 2012 15:20:06 +0000
> > > "Villmow, Micah" <Micah.Villmow at amd.com> wrote:
> > >
> > > > Ping!
> > > >
> > > > > -----Original Message-----
> > > > > From: Villmow, Micah
> > > > > Sent: Tuesday, September 18, 2012 10:33 AM
> > > > > To: Villmow, Micah; Duncan Sands; llvm-commits at cs.uiuc.edu
> > > > > Subject: RE: [llvm-commits] [Patch] Add in expansions for
> > > > > various condition codes
> > > > >
> > > > > Ping!
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-
> > > > > > bounces at cs.uiuc.edu] On Behalf Of Villmow, Micah
> > > > > > Sent: Thursday, September 13, 2012 8:39 AM
> > > > > > To: Duncan Sands; llvm-commits at cs.uiuc.edu
> > > > > > Subject: Re: [llvm-commits] [Patch] Add in expansions for
> > > > > > various condition codes
> > > > > >
> > > > > > txt file attached.
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: llvm-commits-bounces at cs.uiuc.edu [mailto:llvm-commits-
> > > > > > > bounces at cs.uiuc.edu] On Behalf Of Duncan Sands
> > > > > > > Sent: Thursday, September 13, 2012 8:16 AM
> > > > > > > To: llvm-commits at cs.uiuc.edu
> > > > > > > Subject: Re: [llvm-commits] [Patch] Add in expansions for
> > > > > > > various condition codes
> > > > > > >
> > > > > > > Hi Micah,
> > > > > > >
> > > > > > > On 13/09/12 17:09, Villmow, Micah wrote:
> > > > > > > > Updated patch with better comments for fall through.
> > > > > > >
> > > > > > > the patch is attached as some kind of mysterious winmail.dat
> > > > > > > thing. Better to use simple text.
> > > > > > >
> > > > > > > Ciao, Duncan.
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > llvm-commits mailing list
> > > > > > > llvm-commits at cs.uiuc.edu
> > > > > > > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> > > >
> > >
> > >
> > >
> > > --
> > > Hal Finkel
> > > Postdoctoral Appointee
> > > Leadership Computing Facility
> > > Argonne National Laboratory
> >
> >
> 
> 
> 
> --
> Hal Finkel
> Postdoctoral Appointee
> Leadership Computing Facility
> Argonne National Laboratory

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: comparison_expansion_update.txt
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120927/6f8bf073/attachment.txt>


More information about the llvm-commits mailing list