[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2 0/5] Misc cleanups for internal.h & fix tests with CLang



  Daniel P. Berrange wrote:

> When writing the fix for test suite mocking under CLang I found a
> bunch of cruft in internal.h The first four patches thus cleanup
> up internal.h. We then add the extra annotations requird to prevent
> CLang optimizer breaking mock overrides.
> 
> Changed in v2:
> 
>  - Fixed version check to find clang
>  - Use 'printf' instead of 'gnu_printf' on clang still
>  - Addd fix for mock functions under clang
> 
> Daniel P. Berrange (5):
>   Remove duplicate define of __GNUC_PREREQ
>   Require use of GCC 4.4 or CLang compilers
>   Remove network constants out of internal.h
>   Remove incorrectly used TODO macro
>   Prevent more compiler optimization of mockable functions
> 
>  build-aux/mock-noinline.pl             |   2 +-
>  config-post.h                          |  24 ++---
>  src/check-symfile.pl                   |   2 +-
>  src/internal.h                         | 173 ++++++++++-----------------------
>  src/libxl/libxl_conf.c                 |   1 +
>  src/nwfilter/nwfilter_dhcpsnoop.c      |   1 +
>  src/nwfilter/nwfilter_gentech_driver.c |   1 +
>  src/qemu/qemu_capspriv.h               |   2 +-
>  src/qemu/qemu_conf.c                   |   1 +
>  src/rpc/virnetsocket.h                 |   4 +-
>  src/util/vircommand.h                  |   2 +-
>  src/util/vircrypto.h                   |   2 +-
>  src/util/virfile.h                     |   2 +-
>  src/util/virhostcpu.h                  |   4 +-
>  src/util/virmacaddr.h                  |   2 +-
>  src/util/virnetdev.h                   |   8 +-
>  src/util/virnetdevip.h                 |   2 +-
>  src/util/virnetdevopenvswitch.h        |   2 +-
>  src/util/virnetdevtap.h                |   6 +-
>  src/util/virnuma.h                     |  16 +--
>  src/util/virrandom.h                   |   6 +-
>  src/util/virscsi.h                     |   2 +-
>  src/util/virscsivhost.h                |   2 +-
>  src/util/virsocketaddr.h               |  16 +++
>  src/util/virtpm.h                      |   2 +-
>  src/util/virutil.c                     |   1 +
>  src/util/virutil.h                     |  10 +-
>  src/util/viruuid.h                     |   2 +-
>  src/vz/vz_sdk.c                        |   1 +
>  src/xen/xen_hypervisor.c               |   6 +-
>  src/xen/xend_internal.c                |   6 +-
>  31 files changed, 135 insertions(+), 176 deletions(-)

Works fine for me with clang 3.9.1 and clang 4.0.0.

Roman Bogorodskiy

Attachment: signature.asc
Description: PGP signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]