[v1] qemu/bswap: Use compiler __builtin_bswap()

[PATCH 0/4] qemu/bswap: Use compiler __builtin_bswap()

Philippe Mathieu-Daudé posted 4 patches 5 years, 1 month ago

Diff against v2 v3
Download series mbox

Test docker-quick@centos7 passed

Test docker-mingw@fedora passed

Test checkpatch passed

Test FreeBSD passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200923100220.674903-1-philmd@redhat.com

There is a newer version of this series

configure            | 13 ---------
include/qemu/bswap.h | 69 +++++++++-----------------------------------
2 files changed, 13 insertions(+), 69 deletions(-)

Expand all Fold all

[PATCH 0/4] qemu/bswap: Use compiler __builtin_bswap()

Posted by Philippe Mathieu-Daudé 5 years, 1 month ago

Implement Richard's suggestion to use __builtin_bswap().

const_XX() not updated. We could use le_bswap() definitions
but they are undefined, not sure yet what is the best way.
Meanwhile I figure this out, these patches are ready for review.

Philippe Mathieu-Daudé (4):
  qemu/bswap: Remove unused qemu_bswap_len()
  qemu/bswap: Replace bswapXX() by compiler __builtin_bswap()
  qemu/bswap: Replace bswapXXs() by compiler __builtin_bswap()
  qemu/bswap: Remove <byteswap.h> dependency

 configure            | 13 ---------
 include/qemu/bswap.h | 69 +++++++++-----------------------------------
 2 files changed, 13 insertions(+), 69 deletions(-)

-- 
2.26.2