[v1,72/72] mlx5: build pmd only with the clang compiler

Message ID 20201027232335.31427-73-ophirmu@nvidia.com (mailing list archive)
State Superseded, archived
Delegated to: Raslan Darawsheh
Headers
Series mlx5 Windows support - part #5 |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail apply issues

Commit Message

Ophir Munk Oct. 27, 2020, 11:23 p.m. UTC
  From: Tal Shnaiderman <talshn@nvidia.com>

Currently the mlx5 PMD is supported on Windows for clang
compiler only, adding restriction in meson.build files
until mingw support will be added as well.

Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
---
 drivers/common/mlx5/meson.build | 4 ++--
 drivers/net/mlx5/meson.build    | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)
  

Comments

Thomas Monjalon Oct. 28, 2020, 7:35 a.m. UTC | #1
28/10/2020 00:23, Ophir Munk:
> From: Tal Shnaiderman <talshn@nvidia.com>
> 
> Currently the mlx5 PMD is supported on Windows for clang
> compiler only, adding restriction in meson.build files
> until mingw support will be added as well.

I should be squashed with the patch adding Windows compilation.
  

Patch

diff --git a/drivers/common/mlx5/meson.build b/drivers/common/mlx5/meson.build
index f8515d2..3047b45 100644
--- a/drivers/common/mlx5/meson.build
+++ b/drivers/common/mlx5/meson.build
@@ -1,9 +1,9 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright 2019 Mellanox Technologies, Ltd
 
-if not (is_linux or is_windows)
+if not (is_linux or (is_windows and is_ms_linker))
 	build = false
-	reason = 'only supported on Linux and Windows'
+	reason = 'only supported on Linux and Windows build with clang'
 	subdir_done()
 endif
 
diff --git a/drivers/net/mlx5/meson.build b/drivers/net/mlx5/meson.build
index f75a169..a18d08e 100644
--- a/drivers/net/mlx5/meson.build
+++ b/drivers/net/mlx5/meson.build
@@ -2,9 +2,9 @@ 
 # Copyright 2018 6WIND S.A.
 # Copyright 2018 Mellanox Technologies, Ltd
 
-if not (is_linux or is_windows)
+if not (is_linux or (is_windows and is_ms_linker))
 	build = false
-	reason = 'only supported on Linux and Windows'
+	reason = 'only supported on Linux and Windows build with clang'
 	subdir_done()
 endif