On Fri, May 01, 2020 at 11:44:39PM +0100, Stuart Henderson wrote:
> On 2020/04/28 22:11, Jonathan Gray wrote:
> > CVSROOT: /cvs
> > Module name: ports
> > Changes by: [email protected] 2020/04/28 22:11:29
> >
> > Modified files:
> > graphics/vulkan-loader: Makefile
> > graphics/vulkan-loader/patches: patch-loader_CMakeLists_txt
> >
> > Log message:
> > backport a patch to fix build when there is no as(1)
> > fixes build on aarch64
> >
> > ok phessler@ maintainer thfr@ does not have time to review but is fine
> > with this going in
> >
>
> This breaks i386:
> <instantiation>:13:17: error: cannot use more than one symbol in memory
> operand
> push offset termin_error_string@GOT # Push the error string (fourth arg)
> ^
The builtin assembler in clang 8.0 can not handle 'offset' here,
support was added in clang 10.0.
While it should be possible to build with -fno-integrated-as on i386 to
force the use of gas this currently results in not being able to find
the generated gen_defines.asm file. I've not yet been able to convince
cmake to do the right thing with include paths. The file can not be
found even with an explicit -I${WRKBUILD}/loader/.
