| Message ID | 20230518102222.11946-1-ps.report@gmx.net |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [v1] package/llvm: add upstream patch to fix host compile (missing include in Signals.h) | expand |
Peter, All, On 2023-05-18 12:22 +0200, Peter Seiderer spake thusly: > Add upstream patch to fix host compile error on openSUSE Tumbleweed > (gcc-13.0.1). > > Fixes: > > .../build/host-llvm-11.1.0/include/llvm/Support/Signals.h:117:8: error: variable or field ‘CleanupOnSignal’ declared void > 117 | void CleanupOnSignal(uintptr_t Context); > | ^~~~~~~~~~~~~~~ > .../build/host-llvm-11.1.0/include/llvm/Support/Signals.h:117:24: error: ‘uintptr_t’ was not declared in this scope > 117 | void CleanupOnSignal(uintptr_t Context); > | ^~~~~~~~~ > .../build/host-llvm-11.1.0/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 | > > Signed-off-by: Peter Seiderer <ps.report@gmx.net> Applied to master, with a rewritten commit title, thanks. Regards, Yann E. MORIN. > --- > ...-missing-cstdint-header-to-Signals.h.patch | 34 +++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch > > diff --git a/package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch b/package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch > new file mode 100644 > index 0000000000..c798f585d6 > --- /dev/null > +++ b/package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch > @@ -0,0 +1,34 @@ > +From c6c71ab47d658ab8f84e6a6066d476fd9632bbca Mon Sep 17 00:00:00 2001 > +From: Sergei Trofimovich <slyich@gmail.com> > +Date: Mon, 23 May 2022 08:03:23 +0100 > +Subject: [PATCH] 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: https://github.com/llvm/llvm-project/commit/ff1681ddb303223973653f7f5f3f3435b48a1983.patch > +Signed-off-by: Peter Seiderer <ps.report@gmx.net> > +--- > + include/llvm/Support/Signals.h | 1 + > + 1 file changed, 1 insertion(+) > + > +diff --git a/include/llvm/Support/Signals.h b/include/llvm/Support/Signals.h > +index e0a18e72..148216b8 100644 > +--- a/include/llvm/Support/Signals.h > ++++ b/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 { > +-- > +2.40.1 > + > -- > 2.40.1 > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot
>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes: > Add upstream patch to fix host compile error on openSUSE Tumbleweed > (gcc-13.0.1). > Fixes: > .../build/host-llvm-11.1.0/include/llvm/Support/Signals.h:117:8: error: variable or field ‘CleanupOnSignal’ declared void > 117 | void CleanupOnSignal(uintptr_t Context); > | ^~~~~~~~~~~~~~~ > .../build/host-llvm-11.1.0/include/llvm/Support/Signals.h:117:24: error: ‘uintptr_t’ was not declared in this scope > 117 | void CleanupOnSignal(uintptr_t Context); > | ^~~~~~~~~ > .../build/host-llvm-11.1.0/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 | > Signed-off-by: Peter Seiderer <ps.report@gmx.net> Committed to 2023.02.x, thanks.
diff --git a/package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch b/package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch new file mode 100644 index 0000000000..c798f585d6 --- /dev/null +++ b/package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch @@ -0,0 +1,34 @@ +From c6c71ab47d658ab8f84e6a6066d476fd9632bbca Mon Sep 17 00:00:00 2001 +From: Sergei Trofimovich <slyich@gmail.com> +Date: Mon, 23 May 2022 08:03:23 +0100 +Subject: [PATCH] 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: https://github.com/llvm/llvm-project/commit/ff1681ddb303223973653f7f5f3f3435b48a1983.patch +Signed-off-by: Peter Seiderer <ps.report@gmx.net> +--- + include/llvm/Support/Signals.h | 1 + + 1 file changed, 1 insertion(+) + +diff --git a/include/llvm/Support/Signals.h b/include/llvm/Support/Signals.h +index e0a18e72..148216b8 100644 +--- a/include/llvm/Support/Signals.h ++++ b/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 { +-- +2.40.1 +
Add upstream patch to fix host compile error on openSUSE Tumbleweed (gcc-13.0.1). Fixes: .../build/host-llvm-11.1.0/include/llvm/Support/Signals.h:117:8: error: variable or field ‘CleanupOnSignal’ declared void 117 | void CleanupOnSignal(uintptr_t Context); | ^~~~~~~~~~~~~~~ .../build/host-llvm-11.1.0/include/llvm/Support/Signals.h:117:24: error: ‘uintptr_t’ was not declared in this scope 117 | void CleanupOnSignal(uintptr_t Context); | ^~~~~~~~~ .../build/host-llvm-11.1.0/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 | Signed-off-by: Peter Seiderer <ps.report@gmx.net> --- ...-missing-cstdint-header-to-Signals.h.patch | 34 +++++++++++++++++++ 1 file changed, 34 insertions(+) create mode 100644 package/llvm/0002-Add-missing-cstdint-header-to-Signals.h.patch