[PATCH v2 0/3] add diagnostics macros to make code portable

[Andre Muezerie <andremue@linux.microsoft.com>]

It was a common pattern to have "GCC diagnostic ignored" pragmas
sprinkled over the code and only activate these pragmas for certain
compilers (gcc and clang). Clang supports GCC's pragma for
compatibility with existing source code, so #pragma GCC diagnostic
and #pragma clang diagnostic are synonyms for Clang
(https://clang.llvm.org/docs/UsersManual.html).

Now that effort is being made to make the code compatible with MSVC
these expressions would become more complex. It makes sense to hide
this complexity behind macros. This makes maintenance easier as these
macros are defined in a single place. As a plus the code becomes
more readable as well.

v2:
 * Removed __rte_diagnostic_ignored_wstrict_aliasing (introduced
   in v1).
 * Removed the pragmas from many files where they were not needed.
 * In the files where the pragmas were indeed needed, reduced the
   scope during which they are active, reducing the chance that
   unforeseen issues are hidden due to warning suppression.

Andre Muezerie (3):
  lib/eal: add diagnostics macros to make code portable
  drivers/common: add diagnostics macros to make code portable
  drivers/net: add diagnostics macros to make code portable

 drivers/common/idpf/idpf_common_rxtx_avx512.c | 46 +++++++++++++++++--
 drivers/net/axgbe/axgbe_rxtx.h                |  9 ----
 drivers/net/cpfl/cpfl_rxtx_vec_common.h       |  4 --
 drivers/net/dpaa2/dpaa2_rxtx.c                | 16 ++-----
 drivers/net/fm10k/fm10k_rxtx_vec.c            | 19 ++++++--
 drivers/net/hns3/hns3_rxtx_vec_neon.h         |  2 -
 .../net/i40e/i40e_recycle_mbufs_vec_common.c  |  2 -
 drivers/net/i40e/i40e_rxtx_common_avx.h       | 10 ++--
 drivers/net/i40e/i40e_rxtx_vec_altivec.c      |  2 -
 drivers/net/i40e/i40e_rxtx_vec_avx2.c         | 16 +++++--
 drivers/net/i40e/i40e_rxtx_vec_avx512.c       | 16 +++++--
 drivers/net/i40e/i40e_rxtx_vec_common.h       |  4 --
 drivers/net/i40e/i40e_rxtx_vec_neon.c         |  3 --
 drivers/net/i40e/i40e_rxtx_vec_sse.c          | 22 +++++++--
 drivers/net/iavf/iavf_rxtx_vec_avx2.c         | 21 +++++++--
 drivers/net/iavf/iavf_rxtx_vec_avx512.c       | 27 +++++++++--
 drivers/net/iavf/iavf_rxtx_vec_common.h       | 10 ++--
 drivers/net/iavf/iavf_rxtx_vec_sse.c          | 30 ++++++++++--
 drivers/net/ice/ice_rxtx_common_avx.h         | 10 ++--
 drivers/net/ice/ice_rxtx_vec_avx2.c           | 16 +++++--
 drivers/net/ice/ice_rxtx_vec_avx512.c         | 16 +++++--
 drivers/net/ice/ice_rxtx_vec_common.h         |  4 --
 drivers/net/ice/ice_rxtx_vec_sse.c            | 22 +++++++--
 drivers/net/idpf/idpf_rxtx_vec_common.h       |  4 --
 .../ixgbe/ixgbe_recycle_mbufs_vec_common.c    |  2 -
 drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c       |  2 -
 drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c        | 19 ++++++--
 drivers/net/mlx5/mlx5_flow.c                  |  6 +--
 drivers/net/mlx5/mlx5_rxtx_vec_altivec.h      |  5 --
 drivers/net/mlx5/mlx5_rxtx_vec_neon.h         |  2 -
 drivers/net/mlx5/mlx5_rxtx_vec_sse.h          |  4 --
 drivers/net/tap/tap_flow.c                    |  6 +--
 drivers/net/virtio/virtio_rxtx_simple.c       |  4 --
 lib/eal/include/rte_common.h                  | 23 ++++++++++
 34 files changed, 269 insertions(+), 135 deletions(-)

--
2.47.0.vfs.0.3