Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • mesa mesa
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3,033
    • Issues 3,033
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 993
    • Merge requests 993
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • MesaMesa
  • mesamesa
  • Merge requests
  • !17548

d3d12, dzn, CI: Update DirectX-Headers dependency

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Jesse Natalie requested to merge jenatali/mesa:directx-headers-update into main Jul 14, 2022
  • Overview 10
  • Commits 6
  • Pipelines 9
  • Changes 20

A new tag was pushed to DirectX-Headers today which adds some #ifdefs to the API definitions around _WIN32 and _MSC_VER, which enables C++ callers to use non-MSVC compilers to target the MSVC ABI (which was impossible before), and enables C callers to target the WSL ABI (which was possible with ugly casting before).

This series adds some #ifdefs and wrappers to be able to build Gallium d3d12 with the correct ABI for MinGW, and to be able to build Dozen with the correct ABI for WSL without the ugly casts. Since this tag was just pushed today, I'm not going to bump the required dependency... but in the near future I'll do that and clean up some of the preprocessor conditions here. This includes !17544 (closed), or else Dozen will no longer build with these new headers.

The series also updates the container images for Debian and MinGW to pick up the new headers, and locks the wrap reference (which should've been done a long time ago).

/cc @lygstate

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: directx-headers-update