From f421f4cbba62e61f833078cc748791a993d31a69 Mon Sep 17 00:00:00 2001 From: Juan Miguel Carceller Date: Mon, 3 Aug 2026 15:21:59 +0200 Subject: [PATCH 1/4] CI: test with GCC 16 --- .github/workflows/test.yml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 68f5d1549..37278ecc5 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -18,9 +18,10 @@ jobs: fail-fast: false matrix: LCG: ["dev3/x86_64-el9-gcc15-opt", - "dev4/x86_64-el9-clang22-opt", - "dev4/x86_64-el9-gcc15-opt", - "LCG_108/x86_64-el9-gcc15-opt", # root 6.36.02 with c++23 + "dev4/x86_64-el9-clang22-opt", + "dev4/x86_64-el9-gcc15-opt", + "dev4/x86_64-el9-gcc16-opt", + "LCG_108/x86_64-el9-gcc15-opt", # root 6.36.02 with c++23 "LCG_107/x86_64-el9-gcc13-opt", # root 6.34.02 !minimal root version for RNTuple "LCG_104/x86_64-el9-gcc13-opt"] # root 6.28.04 !minimal root version check steps: From d11111c1f7aa62fe716fc6c2b4a8bf333855ba7c Mon Sep 17 00:00:00 2001 From: Thomas Madlener Date: Mon, 24 Aug 2026 11:36:35 +0200 Subject: [PATCH 2/4] Include arrow headers explicitly to fix warning GCC16 starts to speculatively devirtualize (wrongly!) if all of the api header is available which results in warnings that are not suppressable with compiler flags otherwise. --- src/ArrowFrameConverter.cc | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/ArrowFrameConverter.cc b/src/ArrowFrameConverter.cc index c6edbdd0f..d4b5e55c7 100644 --- a/src/ArrowFrameConverter.cc +++ b/src/ArrowFrameConverter.cc @@ -5,8 +5,18 @@ #include "podio/utilities/ArrowTypeRegistry.h" #include "podio/utilities/ArrowUtils.h" -#include -#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + #include namespace podio { From d73d1ddad27010ba3bdd529013064654edc0efb8 Mon Sep 17 00:00:00 2001 From: Juan Miguel Carceller Date: Wed, 26 Aug 2026 10:45:11 +0200 Subject: [PATCH 3/4] Revert "Include arrow headers explicitly to fix warning" This reverts commit d11111c1f7aa62fe716fc6c2b4a8bf333855ba7c. --- src/ArrowFrameConverter.cc | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/src/ArrowFrameConverter.cc b/src/ArrowFrameConverter.cc index d4b5e55c7..c6edbdd0f 100644 --- a/src/ArrowFrameConverter.cc +++ b/src/ArrowFrameConverter.cc @@ -5,18 +5,8 @@ #include "podio/utilities/ArrowTypeRegistry.h" #include "podio/utilities/ArrowUtils.h" -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include - +#include +#include #include namespace podio { From 8c44c7a23a420644a32477f4360aaa86dcadeb6b Mon Sep 17 00:00:00 2001 From: Juan Miguel Carceller Date: Wed, 26 Aug 2026 11:53:02 +0200 Subject: [PATCH 4/4] Reapply "Include arrow headers explicitly to fix warning" This reverts commit d73d1ddad27010ba3bdd529013064654edc0efb8. --- src/ArrowFrameConverter.cc | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/ArrowFrameConverter.cc b/src/ArrowFrameConverter.cc index c6edbdd0f..d4b5e55c7 100644 --- a/src/ArrowFrameConverter.cc +++ b/src/ArrowFrameConverter.cc @@ -5,8 +5,18 @@ #include "podio/utilities/ArrowTypeRegistry.h" #include "podio/utilities/ArrowUtils.h" -#include -#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + #include namespace podio {