[EXTERNAL] Re: [Patch 1/5] rs6000, Add 128-bit sign extension support

will schmidt will_schmidt@vnet.ibm.com
Thu Aug 13 23:53:56 GMT 2020


On Thu, 2020-08-13 at 17:55 -0500, Segher Boessenkool wrote:
> Hi!
> 
> On Thu, Aug 13, 2020 at 05:11:11PM -0500, will schmidt wrote:
> > > > That is probably a level of detail that is not
> > > > really needed in the GCC code comment.  Probably best to just
> > > > change
> > > > the comment to read something like "ISA 3.0 sign extend
> > > > builtins". 
> > > 
> > > Sounds good.
> > 
> > As long as there are no issues defining the builtins for 3.0 here.
> > AFAIK they are not documented in ISA 3.0.  This is a happy accident
> > that these ISA 3.1 builtins can be implemented with existing
> > support.
> 
> There are *no* builtins defined in the ISA!  The insns are just ISA
> 3.0
> instructions.
> 

Ok. 

So then maybe just "Sign extend builtins" and leave off the ISA
reference all together.   

:-)

thanks
-WIll

> 
> Segher



More information about the Gcc-patches mailing list