[PATCH] [mips][msa] Removed fcge, fcgt, fsge, fsgt

[Daniel Sanders]

I agree. I've updated the patch with a better commit message.

> -----Original Message-----
> From: Hal Finkel [mailto:hfinkel at anl.gov]
> Sent: 19 August 2013 10:11
> To: Daniel Sanders
> Cc: llvm-commits at cs.uiuc.edu; Jack Carter
> Subject: Re: [PATCH] [mips][msa] Removed fcge, fcgt, fsge, fsgt
> 
> ----- Original Message -----
> > Hi,
> >
> > There's no existing hardware that supported these instructions. Some
> > of our unreleased software models supported them but these have also
> > been updated to the match the published version of the MSA spec.
> 
> Sounds good. Personally, I think that you should note that somehow in the
> commit log (maybe call it a draft spec) just so that bystanders understand
> what's going on.
> 
>  -Hal
> 
> >
> > > -----Original Message-----
> > > From: Hal Finkel [mailto:hfinkel at anl.gov]
> > > Sent: 16 August 2013 17:46
> > > To: Daniel Sanders
> > > Cc: llvm-commits at cs.uiuc.edu; Jack Carter
> > > Subject: Re: [PATCH] [mips][msa] Removed fcge, fcgt, fsge, fsgt
> > >
> > > ----- Original Message -----
> > > >
> > > >
> > > >
> > > >
> > > > Removed support for fcge, fcgt, fsge, and fsgt. These were present
> > > > in older versions of the spec but were removed in r1.04.
> > >
> > > Out of curiosity, are you removing support for instructions that are
> > > implemented in current in-production hardware?
> > >
> > >  -Hal
> > >
> > > >
> > > >
> > > >
> > > > Daniel Sanders
> > > >
> > > > Leading Software Design Engineer, MIPS Processor IP
> > > >
> > > > Imagination Technologies Limited
> > > >
> > > > www.imgtec.com
> > > >
> > > >
> > > > _______________________________________________
> > > > llvm-commits mailing list
> > > > llvm-commits at cs.uiuc.edu
> > > > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> > > >
> > >
> > > --
> > > Hal Finkel
> > > Assistant Computational Scientist
> > > Leadership Computing Facility
> > > Argonne National Laboratory
> >
> 
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-mips-msa-Removed-fcge-fcgt-fsge-fsgt.patch
Type: application/octet-stream
Size: 14774 bytes
Desc: 0004-mips-msa-Removed-fcge-fcgt-fsge-fsgt.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130819/829ae023/attachment.obj>