FWIW: the uninative with gcc-13 means only that it is compatible with
gcc-13 on host, it doesn't include gcc-native (which doesn't exist).

I'm using already newer rust with kirkstone, so I cannot confirm this issue
with gcc-13 on host, but I'm not so surprised if it wasn't caught on AB yet.

Regards,

On Tue, Jul 11, 2023 at 9:05 AM A. Sverdlin via lists.openembedded.org
<[email protected]> wrote:

> Hello Jose,
>
> On Mon, 2023-07-10 at 23:58 +0100, Jose Quaresma wrote:
> > Hi Alexander,
> >
> > The uninative release with the gcc 13 was released a few weeks ago,
> > what is the reason for this issue not being found on the autobuild when
> using the uninative?
>
> I'm not sure why it wasn't found by (and what actually is being meant as)
> autobuild, but the issue is real:
>
> DEBUG: Executing shell function do_compile
> NOTE: VERBOSE=1 cmake --build
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/build --target all --
> [1/2283] .../tmp/hosttools/g++ -DGTEST_HAS_RTTI=0 -D_GNU_SOURCE
> -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-
> r0/build/utils/TableGen
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/utils/TableGen
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-
> r0/build/include
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/include
> -isystem.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/recipe-sysroot-
> native/usr/include -O2 -pipe -fPIC -fno-semantic-interposition
> -fvisibility-inlines-hidden -Werror=date-time -Wall -Wextra
> -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wno-missing-field-
> initializers -pedantic -Wno-long-long -Wimplicit-fallthrough
> -Wno-maybe-uninitialized -Wno-class-memaccess -Wno-redundant-move
> -Wno-pessimizing-move -Wno-noexcept-type -Wdelete-non-virtual-dtor -
> Wsuggest-override -Wno-comment -Wmisleading-indentation
> -fdiagnostics-color -ffunction-sections -fdata-sections -DNDEBUG
> -fno-exceptions -fno-rtti -std=c++14 -MD -MT utils/TableGen/CMakeFiles/llvm-
> tblgen.dir/Attributes.cpp.o -MF
> utils/TableGen/CMakeFiles/llvm-tblgen.dir/Attributes.cpp.o.d -o
> utils/TableGen/CMakeFiles/llvm-tblgen.dir/Attributes.cpp.o -c
> .../tmp/work/x86_64-linux/rust-llvm-
>
> native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/utils/TableGen/Attributes.cpp
> [2/2283] .../tmp/hosttools/g++ -DGTEST_HAS_RTTI=0 -D_GNU_SOURCE
> -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-
> r0/build/lib/Support
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/lib/Support
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/build/include -
> I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/include
> -isystem.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/recipe-sysroot-native/usr/include
> -O2
> -pipe -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden
> -Werror=date-time -Wall -Wextra -Wno-unused-parameter -Wwrite-strings
> -Wcast-qual -Wno-missing-field-initializers -pedantic -Wno-
> long-long -Wimplicit-fallthrough -Wno-maybe-uninitialized
> -Wno-class-memaccess -Wno-redundant-move -Wno-pessimizing-move
> -Wno-noexcept-type -Wdelete-non-virtual-dtor -Wsuggest-override
> -Wno-comment -
> Wmisleading-indentation -fdiagnostics-color -ffunction-sections
> -fdata-sections -DNDEBUG -std=c++14  -fno-exceptions -fno-rtti -MD -MT
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o -MF
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o.d -o
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o -c
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-
> project/llvm/lib/Support/Signals.cpp
> FAILED: lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o
> .../tmp/hosttools/g++ -DGTEST_HAS_RTTI=0 -D_GNU_SOURCE
> -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/build/lib/Support -
> I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/lib/Support
> -I.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/build/include
> -I.../tmp/work/x86_64-
> linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/include
> -isystem.../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/recipe-sysroot-native/usr/include
> -O2 -pipe -fPIC -fno-
> semantic-interposition -fvisibility-inlines-hidden -Werror=date-time -Wall
> -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual
> -Wno-missing-field-initializers -pedantic -Wno-long-long -
> Wimplicit-fallthrough -Wno-maybe-uninitialized -Wno-class-memaccess
> -Wno-redundant-move -Wno-pessimizing-move -Wno-noexcept-type
> -Wdelete-non-virtual-dtor -Wsuggest-override -Wno-comment -Wmisleading-
> indentation -fdiagnostics-color -ffunction-sections -fdata-sections
> -DNDEBUG -std=c++14  -fno-exceptions -fno-rtti -MD -MT
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o -MF
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o.d -o
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o -c
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-
> project/llvm/lib/Support/Signals.cpp
> In file included from
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/lib/Support/Signals.cpp:14:
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/include/llvm/Support/Signals.h:119:8:
> error: variable or field ‘CleanupOnSignal’ declared void
>   119 |   void CleanupOnSignal(uintptr_t Context);
>       |        ^~~~~~~~~~~~~~~
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/include/llvm/Support/Signals.h:119:24:
> error: ‘uintptr_t’ was not declared in this scope
>   119 |   void CleanupOnSignal(uintptr_t Context);
>       |                        ^~~~~~~~~
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/include/llvm/Support/Signals.h:18:1:
> note: ‘uintptr_t’ is defined in header ‘<cstdint>’; did you forget to
> ‘#include <cstdint>’?
>    17 | #include <string>
>   +++ |+#include <cstdint>
>    18 |
> In file included from
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/lib/Support/Signals.cpp:251:
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/rustc-1.59.0-src/src/llvm-project/llvm/lib/Support/Unix/Signals.inc:348:44:
> error: ‘void llvm::sys::CleanupOnSignal(uintptr_t)’ should have been
> declared inside ‘llvm::sys’
>   348 | void sys::CleanupOnSignal(uintptr_t Context) {
>       |                                            ^
>
> ...
>
> ninja: build stopped: subcommand failed.
> WARNING:
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/temp/run.do_compile.556945:162
> exit 1 from 'eval ${DESTDIR:+DESTDIR=${DESTDIR} }VERBOSE=1 cmake --build
> '.../tmp/work/x86_64-linux/rust-
> llvm-native/1.59.0-r0/build' "$@" -- ${EXTRA_OECMAKE_BUILD}'
> WARNING: Backtrace (BB generated script):
>         #1: cmake_runcmake_build,
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/temp/run.do_compile.556945,
> line 162
>         #2: cmake_do_compile,
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/temp/run.do_compile.556945,
> line 156
>         #3: do_compile,
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/temp/run.do_compile.556945,
> line 151
>         #4: main,
> .../tmp/work/x86_64-linux/rust-llvm-native/1.59.0-r0/temp/run.do_compile.556945,
> line 175
>
>
> > With the clang toolchain this issue [1] comes right after the uninative
> with gcc-13 was released.
> > [1] https://github.com/kraj/meta-clang/pull/816
> >
> > Jose
> >
> > A. Sverdlin via lists.openembedded.org <alexander.sverdlin=
> [email protected]> escreveu no dia segunda, 10/07/2023
> à(s) 18:15:
> > > From: Alexander Sverdlin <[email protected]>
> > >
> > > * needed for rust-llvm-native on hosts with gcc-13
> > >
> > > Based on commit 3382759cb6c5 ("llvm: backport a fix for build with
> gcc-13")
> > >
> > > Signed-off-by: Alexander Sverdlin <[email protected]>
> > > ---
> > > Changelog:
> > > v2: rebase onto 2022-04.11-kirkstone
> > >
> > >  meta/recipes-devtools/rust/rust-llvm.inc      |  4 ++-
> > >  ...-missing-cstdint-header-to-Signals.h.patch | 32 +++++++++++++++++++
> > >  2 files changed, 35 insertions(+), 1 deletion(-)
> > >  create mode 100644
> meta/recipes-devtools/rust/rust-llvm/0003-Support-Add-missing-cstdint-header-to-Signals.h.patch
> > >
> > > diff --git a/meta/recipes-devtools/rust/rust-llvm.inc
> b/meta/recipes-devtools/rust/rust-llvm.inc
> > > index 5c2ccdac9a..e645e7a7ac 100644
> > > --- a/meta/recipes-devtools/rust/rust-llvm.inc
> > > +++ b/meta/recipes-devtools/rust/rust-llvm.inc
> > > @@ -3,7 +3,9 @@ LICENSE ?= "Apache-2.0-with-LLVM-exception"
> > >  HOMEPAGE = "http://www.rust-lang.org";
> > >
> > >  SRC_URI +=
> "file://0002-llvm-allow-env-override-of-exe-path.patch;striplevel=2 \
> > > -
> file://0001-AsmMatcherEmitter-sort-ClassInfo-lists-by-name-as-we.patch;striplevel=2"
> > > +
> file://0001-AsmMatcherEmitter-sort-ClassInfo-lists-by-name-as-we.patch;striplevel=2
> \
> > > +
> file://0003-Support-Add-missing-cstdint-header-to-Signals.h.patch;striplevel=2
> \
> > > +"
> > >
> > >  S = "${RUSTSRC}/src/llvm-project/llvm"
> > >
> > > diff --git
> a/meta/recipes-devtools/rust/rust-llvm/0003-Support-Add-missing-cstdint-header-to-Signals.h.patch
> b/meta/recipes-devtools/rust/rust-llvm/0003-Support-Add-missing-cstdint-header-to-
> > > Signals.h.patch
> > > new file mode 100644
> > > index 0000000000..6ed23aa9c5
> > > --- /dev/null
> > > +++
> b/meta/recipes-devtools/rust/rust-llvm/0003-Support-Add-missing-cstdint-header-to-Signals.h.patch
> > > @@ -0,0 +1,32 @@
> > > +From a94bf34221fc4519bd8ec72560c2d363ffe2de4c Mon Sep 17 00:00:00 2001
> > > +From: Sergei Trofimovich <[email protected]>
> > > +Date: Mon, 23 May 2022 08:03:23 +0100
> > > +Subject: [PATCH] [Support] Add missing <cstdint> header to Signals.h
> > > +
> > > +Without the change llvm build fails on this week's gcc-13 snapshot as:
> > > +
> > > +    [  0%] Building CXX object
> lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o
> > > +    In file included from llvm/lib/Support/Signals.cpp:14:
> > > +    llvm/include/llvm/Support/Signals.h:119:8: error: variable or
> field 'CleanupOnSignal' declared void
> > > +      119 |   void CleanupOnSignal(uintptr_t Context);
> > > +          |        ^~~~~~~~~~~~~~~
> > > +
> > > +Upstream-Status: Backport [llvmorg-15.0.0
> ff1681ddb303223973653f7f5f3f3435b48a1983]
> > > +Signed-off-by: Martin Jansa <[email protected]>
> > > +Signed-off-by: Alexander Sverdlin <[email protected]>
> > > +---
> > > + llvm/include/llvm/Support/Signals.h | 1 +
> > > + 1 file changed, 1 insertion(+)
> > > +
> > > +diff --git a/llvm/include/llvm/Support/Signals.h
> b/llvm/include/llvm/Support/Signals.h
> > > +index 44f5a750ff5c..937e0572d4a7 100644
> > > +--- a/llvm/include/llvm/Support/Signals.h
> > > ++++ b/llvm/include/llvm/Support/Signals.h
> > > +@@ -14,6 +14,7 @@
> > > + #ifndef LLVM_SUPPORT_SIGNALS_H
> > > + #define LLVM_SUPPORT_SIGNALS_H
> > > +
> > > ++#include <cstdint>
> > > + #include <string>
> > > +
> > > + namespace llvm {
>
> --
> Alexander Sverdlin
> Siemens AG
> www.siemens.com
>
> 
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#184118): 
https://lists.openembedded.org/g/openembedded-core/message/184118
Mute This Topic: https://lists.openembedded.org/mt/100062248/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to