Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 28 Apr 2025 11:32:44 -0400
From:      John Baldwin <jhb@FreeBSD.org>
To:        Dimitry Andric <dim@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: a34d2231fa38 - main - Partially revert 6527682ab705 for llvm-based projects
Message-ID:  <053e6747-445d-41b5-9117-7d7aa05073e1@FreeBSD.org>
In-Reply-To: <202504241436.53OEanKM076888@gitrepo.freebsd.org>
References:  <202504241436.53OEanKM076888@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 4/24/25 10:36, Dimitry Andric wrote:
> The branch main has been updated by dim:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=a34d2231fa38a5e440ecdc94dbca3b3deb5a7f50
> 
> commit a34d2231fa38a5e440ecdc94dbca3b3deb5a7f50
> Author:     Dimitry Andric <dim@FreeBSD.org>
> AuthorDate: 2025-04-24 12:03:38 +0000
> Commit:     Dimitry Andric <dim@FreeBSD.org>
> CommitDate: 2025-04-24 14:36:40 +0000
> 
>      Partially revert 6527682ab705 for llvm-based projects
>      
>      Upstream llvm compiles most of their subprojects with -std=c++17
>      explicitly, not -std=gnu++17. In gnu++17 mode, both clang and gcc define
>      the macro 'i386' on i386, which clashes with a namespace identifier in
>      contrib/llvm-project/llvm/include/llvm/ExecutionEngine/JITLink/i386.h.
>      
>      Since the default after 6527682ab705 is now CXXSTD=gnu++17, explicitly
>      set CXXSTD=c++17 for lib/clang, lib/libclang_rt and lib/libomp.

Oops.  I had done my testing in base with CXXSTD=c++17, but switched to gnu++17
due to breakage in ports. :(

-- 
John Baldwin




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?053e6747-445d-41b5-9117-7d7aa05073e1>