[PATCH] powerpc: Remove uses of operand modifier (%s) in inline asm

Gabriel F. T. Gomes gftg@linux.vnet.ibm.com
Fri Jan 29 18:14:00 GMT 2016


On Wed, 27 Jan 2016 17:19:12 +0000
Joseph Myers <joseph@codesourcery.com> wrote:

> On Wed, 27 Jan 2016, Gabriel F. T. Gomes wrote:
> 
> > Thus, we can replace __builtin_ffs with __builtin_clz and remove
> > the %s operand modifier.  
> 
> Doesn't that mean you need to make these definitions conditional on 
> __GNUC_PREREQ (3, 4) (which I think should be fine to do - we
> shouldn't need to care about these optimizations for pre-3.4
> compilers), as that was the version where __builtin_clz was
> introduced?
> 

I agree.

The attached patch addresses your concerns.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-powerpc-Remove-uses-of-operand-modifier-s-in-inline-.patch
Type: text/x-patch
Size: 3253 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20160129/cf0c1839/attachment.bin>


More information about the Libc-alpha mailing list