Date: Thu, 19 Nov 2020 13:38:20 +0100 From: "Kristof Provost" <kp@FreeBSD.org> To: "Dimitry Andric" <dim@FreeBSD.org> Cc: freebsd-riscv@freebsd.org, toolchain@freebsd.org Subject: Re: devel/glib20 build issue Message-ID: <7A8DF954-E8B9-4623-85F1-AF49C1A7059D@FreeBSD.org> In-Reply-To: <3BC510E6-F7FC-4341-B715-66472F3640BF@FreeBSD.org> References: <54177870-E411-4DF1-901E-94FE7E5F553F@FreeBSD.org> <3BC510E6-F7FC-4341-B715-66472F3640BF@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 19 Nov 2020, at 11:59, Dimitry Andric wrote: > On 19 Nov 2020, at 11:57, Kristof Provost <kp@freebsd.org> wrote: >> >> While trying to build assorted ports on RISC-V I found that >> devel/glib20 doesn’t build. >> It turns out to be due to a missing >> __GCC_HAVE_SYNC_COMPARE_AND_SWAP_4 define. That define is implicitly >> set by the toolchain. Or, at least, it is on other platforms (ARM, >> MIPS, x86, …), but not on RISC-V. >> >> That’s likely just been forgotten. The following toolchain patch >> fixes that for me: >> >> diff --git a/contrib/llvm-project/clang/lib/Basic/Targets/RISCV.cpp >> b/contrib/llvm-project/clang/lib/Basic/Targets/RISCV.cpp >> index 4ba703c8dd1..73f1b5bc446 100644 >> --- a/contrib/llvm-project/clang/lib/Basic/Targets/RISCV.cpp >> +++ b/contrib/llvm-project/clang/lib/Basic/Targets/RISCV.cpp >> @@ -115,8 +115,14 @@ void RISCVTargetInfo::getTargetDefines(const >> LangOptions &Opts, >> Builder.defineMacro("__riscv_muldiv"); >> } >> >> - if (HasA) >> + if (HasA) { >> Builder.defineMacro("__riscv_atomic"); >> + Builder.defineMacro("__GCC_HAVE_SYNC_COMPARE_AND_SWAP_1"); >> + Builder.defineMacro("__GCC_HAVE_SYNC_COMPARE_AND_SWAP_2"); >> + Builder.defineMacro("__GCC_HAVE_SYNC_COMPARE_AND_SWAP_4"); >> + if (Is64Bit) >> + >> Builder.defineMacro("__GCC_HAVE_SYNC_COMPARE_AND_SWAP_8"); >> + } >> >> if (HasF || HasD) { >> Builder.defineMacro("__riscv_flen", HasD ? "64" : "32"); > > Hi Kristof, > > Can you please submit this upstream? We already have way too many > gratuitous RISC-V differences that haven't been upstreamed. > https://reviews.llvm.org/D91784 Best regards, Kristof
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7A8DF954-E8B9-4623-85F1-AF49C1A7059D>