[Qemu-commits] [qemu/qemu] d783f7: target-mips: Stop using uint_fast*_t types in r4k_...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d783f78933b212537ece77c7ec66866cc2bc0f4d
      
https://github.com/qemu/qemu/commit/d783f78933b212537ece77c7ec66866cc2bc0f4d
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M target-mips/cpu.h

  Log Message:
  -----------
  target-mips: Stop using uint_fast*_t types in r4k_tlb_t struct

The r4k_tlb_t structure uses the uint_fast*_t types. Most of these
uses are in bitfields and are thus pointless, because the bitfield
itself specifies the width of the type; just use 'unsigned int'
instead. (On glibc uint_fast16_t is defined as either 32 or 64 bits,
so we know the code is not reliant on it being exactly 16 bits.)
There is also one use of uint_fast8_t, which we replace with uint8_t,
because both are exactly 8 bits on glibc and this is the only
place outside the softfloat code which uses an int_fast*_t type.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Aurelien Jarno <address@hidden>


  Commit: 0bb721d7217ed4a1abb44f521c5c7ec185062d58
      
https://github.com/qemu/qemu/commit/0bb721d7217ed4a1abb44f521c5c7ec185062d58
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M fpu/softfloat.c
    M include/fpu/softfloat.h

  Log Message:
  -----------
  fpu: Remove use of int_fast16_t in conversions to int16

Make the functions which convert floating point to 16 bit integer
return int16_t rather than int_fast16_t, and correspondingly use
int_fast16_t in their internal implementations where appropriate.

(These functions are used only by the ARM target.)

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Aurelien Jarno <address@hidden>
Message-id: address@hidden


  Commit: 07d792d2b08669bf6a97cbf590496078c4621068
      
https://github.com/qemu/qemu/commit/07d792d2b08669bf6a97cbf590496078c4621068
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M fpu/softfloat-macros.h
    M fpu/softfloat.c

  Log Message:
  -----------
  fpu: Use plain 'int' rather than 'int_fast16_t' for shift counts

Use the plain 'int' type rather than 'int_fast16_t' for shift counts
in the various shift related functions, since we don't actually care
about the size of the integer at all here, and using int16_t would
be confusing.

This should be a safe change because int_fast16_t semantics
permit use of 'int' (and on 32-bit glibc that is what you get).

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Aurelien Jarno <address@hidden>
Message-id: address@hidden


  Commit: 0c48262d4772d40677364199372fb6ffcf487558
      
https://github.com/qemu/qemu/commit/0c48262d4772d40677364199372fb6ffcf487558
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M fpu/softfloat-macros.h
    M fpu/softfloat.c

  Log Message:
  -----------
  fpu: Use plain 'int' rather than 'int_fast16_t' for exponents

Use the plain 'int' type rather than 'int_fast16_t' for handling
exponents. Exponents don't need to be exactly 16 bits, so using int16_t
for them would confuse more than it clarified.

This should be a safe change because int_fast16_t semantics
permit use of 'int' (and on 32-bit glibc that is what you get).

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Aurelien Jarno <address@hidden>
Message-id: address@hidden


  Commit: 50fe4df8ee6aba63ae51457bad40ba26e3c9746f
      
https://github.com/qemu/qemu/commit/50fe4df8ee6aba63ae51457bad40ba26e3c9746f
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M include/qemu/osdep.h

  Log Message:
  -----------
  osdep.h: Remove int_fast*_t Solaris compatibility code

We now do not use the int_fast*_t types anywhere in QEMU, so we can
remove the compatibility definitions we were providing for the
benefit of ancient Solaris versions.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Aurelien Jarno <address@hidden>
Message-id: address@hidden


  Commit: 1badb5869831de916792628b5e159176f7e342b8
      
https://github.com/qemu/qemu/commit/1badb5869831de916792628b5e159176f7e342b8
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Add section for FPU emulation

Add an entry to the MAINTAINERS file for our softfloat FPU
emulation code. This code is only 'odd fixes' but it's useful to
record who to cc on patches to it.

Signed-off-by: Peter Maydell <address@hidden>
Message-id: address@hidden


  Commit: 3ba32c100a51fcfd0ff367a5c40d4e84e206dd3a
      
https://github.com/qemu/qemu/commit/3ba32c100a51fcfd0ff367a5c40d4e84e206dd3a
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-19 (Fri, 19 Feb 2016)

  Changed paths:
    M MAINTAINERS
    M fpu/softfloat-macros.h
    M fpu/softfloat.c
    M include/fpu/softfloat.h
    M include/qemu/osdep.h
    M target-mips/cpu.h

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/pmaydell/tags/pull-softfloat-20160219' 
into staging

softfloat queue:
 * update MAINTAINERS with a section for softfloat
 * drop all the uses of int_fast*_t types

# gpg: Signature made Fri 19 Feb 2016 16:34:35 GMT using RSA key ID 14360CDE
# gpg: Good signature from "Peter Maydell <address@hidden>"
# gpg:                 aka "Peter Maydell <address@hidden>"
# gpg:                 aka "Peter Maydell <address@hidden>"

* remotes/pmaydell/tags/pull-softfloat-20160219:
  MAINTAINERS: Add section for FPU emulation
  osdep.h: Remove int_fast*_t Solaris compatibility code
  fpu: Use plain 'int' rather than 'int_fast16_t' for exponents
  fpu: Use plain 'int' rather than 'int_fast16_t' for shift counts
  fpu: Remove use of int_fast16_t in conversions to int16
  target-mips: Stop using uint_fast*_t types in r4k_tlb_t struct

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/1b3337bb1d1c...3ba32c100a51