[Qemu-devel] [6834] tcg: update README wrt recent bswap changes

[Qemu-devel] [6834] tcg: update README wrt recent bswap changes

[Aurelien Jarno]

Revision: 6834
          http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6834
Author:   aurel32
Date:     2009-03-13 09:35:26 +0000 (Fri, 13 Mar 2009)
Log Message:
-----------
tcg: update README wrt recent bswap changes

Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

Modified Paths:
--------------
    trunk/tcg/README

Modified: trunk/tcg/README
===================================================================
--- trunk/tcg/README	2009-03-13 09:35:19 UTC (rev 6833)
+++ trunk/tcg/README	2009-03-13 09:35:26 UTC (rev 6834)
@@ -263,16 +263,17 @@
 
 8, 16 or 32 bit sign/zero extension (both operands must have the same type)
 
-* bswap16_i32 t0, t1
+* bswap16_i32/i64 t0, t1
 
-16 bit byte swap on a 32 bit value. The two high order bytes must be set
-to zero.
+16 bit byte swap on a 32/64 bit value. The two/six high order bytes must be
+set to zero.
 
-* bswap_i32 t0, t1
+* bswap32_i32/i64 t0, t1
 
-32 bit byte swap
+32 bit byte swap on a 32/64 bit value. With a 32 bit value, the four high
+order bytes must be set to zero.
 
-* bswap_i64 t0, t1
+* bswap64_i64 t0, t1
 
 64 bit byte swap
 

Re: [Qemu-devel] [6834] tcg: update README wrt recent bswap changes

[Stuart Brady]

On Fri, Mar 13, 2009 at 09:35:27AM +0000, Aurelien Jarno wrote:
> -* bswap_i32 t0, t1
> +* bswap32_i32/i64 t0, t1
> 
> -32 bit byte swap
> +32 bit byte swap on a 32/64 bit value. With a 32 bit value, the four high
> +order bytes must be set to zero.
                                                 ^^ 64-bit?

(I think it'd be cheating to only allow byteswaps of 0! ;-))

Otherwise, nice additions, BTW. :-)

Cheers,
-- 
Stuart Brady

Re: [Qemu-devel] [6834] tcg: update README wrt recent bswap changes

[Aurelien Jarno]

On Sat, Mar 14, 2009 at 12:20:49AM +0000, Stuart Brady wrote:
> On Fri, Mar 13, 2009 at 09:35:27AM +0000, Aurelien Jarno wrote:
> > -* bswap_i32 t0, t1
> > +* bswap32_i32/i64 t0, t1
> > 
> > -32 bit byte swap
> > +32 bit byte swap on a 32/64 bit value. With a 32 bit value, the four high
> > +order bytes must be set to zero.
>                                                  ^^ 64-bit?
> 
> (I think it'd be cheating to only allow byteswaps of 0! ;-))
> 

Thanks, fixed.

-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net