Opened 19 months ago

Closed 6 months ago

#9983 closed defect (fixed)

Build error with clang on 32 bit Windows

Reported by: Christoph Reiter Owned by:
Priority: normal Component: avutil
Version: git-master Keywords: vulkan
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no


Looks like clang 15 got stricter somehow:

C:/msys64/home/user/M/mingw-w64-ffmpeg/src/ffmpeg-4.4.2/libavutil/hwcontext_vulkan.c:865:15: error: incompatible pointer to integer conversion assigning to
      'VkCommandPool' (aka 'unsigned long long') from 'void *' [-Wint-conversion]
    cmd->pool = NULL;

Some searching suggests that VkCommandPool is a handle, and that VK_NULL_HANDLE should be used, but I don't know enough about vulkan. It builds fine with it at least.

Attachments (1)

0003-fix-clang32-build.patch (394 bytes ) - added by Christoph Reiter 19 months ago.

Download all attachments as: .zip

Change History (7)

by Christoph Reiter, 19 months ago

comment:1 by Carl Eugen Hoyos, 19 months ago

Component: undeterminedavutil
Keywords: vulkan added
Version: unspecifiedgit-master

Please either provide the configure line you tested and clang -v output to make this a valid ticket or - much better - send your patch made with git format-patch to the FFmpeg development mailing list.

comment:2 by Christoph Reiter, 19 months ago

The configure line is a simple "./configure".

$ clang -v
clang version 15.0.3
Target: i686-w64-windows-gnu
Thread model: posix
InstalledDir: C:/msys64/clang32/bin

I will try to look into providing a patch. I need to read up on the vulkan API first though.

comment:3 by Christoph Reiter, 15 months ago

comment:5 by Sam James, 6 months ago

Resolution: fixed
Status: newclosed

comment:6 by Christoph Reiter, 6 months ago

Resolution: fixed
Status: newclosed

Yeah, this can be closed.

It regressed again in the meantime, but that's another issue, and I'll try to upstream that some time:

Note: See TracTickets for help on using tickets.