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 
> <[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 (#184117): 
https://lists.openembedded.org/g/openembedded-core/message/184117
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