mbox series

[v2,0/2] make dpdk buildable with latest msvc

Message ID 1697550563-17532-1-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
Headers
Series make dpdk buildable with latest msvc |

Message

Tyler Retzlaff Oct. 17, 2023, 1:49 p.m. UTC 
  This series makes some minor temporary modifications to the MSVC build
to allow DPDK to be buildable. The changes are only temporary to allow
Windows/MSVC CI pipeline to be established.

Warnings will be made clean in a future series and avoidance of STDC
atomics check will be removed when the publicly available compiler is
formally released.

With this series applied you can do the following from a
"Developer Command Prompt for VS 2022 Preview" with
Visual Studio 17.8.0 Preview 3.0 installed.

  meson setup --werror -Denable_stdatomic=true b
  meson compile -C b

v2:
    * provide comments indicating which warnings are being disabled

Tyler Retzlaff (2):
  build: temporarily disable MSVC warnings
  eal: disable standard c atomics support check for MSVC

 config/meson.build              | 5 +++++
 lib/eal/include/rte_stdatomic.h | 2 ++
 2 files changed, 7 insertions(+)

Comments

David Marchand Oct. 31, 2023, 6:39 p.m. UTC | #1 
On Tue, Oct 17, 2023 at 3:49 PM Tyler Retzlaff
<roretzla@linux.microsoft.com> wrote:
>
> This series makes some minor temporary modifications to the MSVC build
> to allow DPDK to be buildable. The changes are only temporary to allow
> Windows/MSVC CI pipeline to be established.
>
> Warnings will be made clean in a future series and avoidance of STDC
> atomics check will be removed when the publicly available compiler is
> formally released.
>
> With this series applied you can do the following from a
> "Developer Command Prompt for VS 2022 Preview" with
> Visual Studio 17.8.0 Preview 3.0 installed.
>
>   meson setup --werror -Denable_stdatomic=true b
>   meson compile -C b
>
> v2:
>     * provide comments indicating which warnings are being disabled
>
> Tyler Retzlaff (2):
>   build: temporarily disable MSVC warnings
>   eal: disable standard c atomics support check for MSVC
>
>  config/meson.build              | 5 +++++
>  lib/eal/include/rte_stdatomic.h | 2 ++
>  2 files changed, 7 insertions(+)
>

Series applied, thanks.