Hi,

Sorry for the delay

On Tue, 24 Sept 2024 at 15:17, Jason Merrill <ja...@redhat.com> wrote:
>
> On 9/23/24 2:08 AM, ci_not...@linaro.org wrote:
> > Dear contributor, our automatic CI has detected problems related to your 
> > patch(es).  Please find some details below.  If you have any questions, 
> > please follow up on linaro-toolchain@lists.linaro.org mailing list, 
> > Libera's #linaro-tcwg channel, or ping your favourite Linaro toolchain 
> > developer on the usual project channel.
> >
> > We understand that it might be difficult to find the necessary logs or 
> > reproduce the issue locally. If you can't get what you need from our CI 
> > within minutes, let us know and we will be happy to help.
> >
> > In gcc_check master-arm after:
> >
> >    | gcc patch https://patchwork.sourceware.org/patch/97864
> >    | Author: Jason Merrill <ja...@redhat.com>
> >    | Date:   Mon Sep 23 11:06:11 2024 -0400
> >    |
> >    |     libstdc++: #ifdef out #pragma GCC system_header
> >    |
> >    |     Tested x86_64-pc-linux-gnu, OK for trunk?
> >    |
> >    |     -- 8< --
> >    |
> >    |     In r15-3714-gd3a7302ec5985a I added -Wsystem-headers to the 
> > libstdc++ build
> >    | ... 379 lines of the commit log omitted.
> >    | ... applied on top of baseline commit:
> >    | 4700ad1c78c c++: diagnose this specifier in requires expr [PR116798]
> >
> > FAIL: 3 regressions
> >
> > regressions.sum:
> >               === libstdc++ tests ===
> >
> > Running libstdc++:libstdc++-dg/conformance.exp ...
> > FAIL: 17_intro/headers/c++2011/parallel_mode.cc (test for excess errors)
> > FAIL: 17_intro/headers/c++2014/parallel_mode.cc (test for excess errors)
> > FAIL: 17_intro/headers/c++2017/parallel_mode.cc -std=gnu++17 (test for 
> > excess errors)
> >
> >
> > You can find the failure logs in *.log.1.xz files in
> >   - 
> > https://ci.linaro.org/job/tcwg_gcc_check--master-arm-precommit/9301/artifact/artifacts/artifacts.precommit/00-sumfiles/
> > The full lists of regressions and improvements as well as configure and 
> > make commands are in
> >   - 
> > https://ci.linaro.org/job/tcwg_gcc_check--master-arm-precommit/9301/artifact/artifacts/artifacts.precommit/notify/
> > The list of [ignored] baseline and flaky failures are in
> >   - 
> > https://ci.linaro.org/job/tcwg_gcc_check--master-arm-precommit/9301/artifact/artifacts/artifacts.precommit/sumfiles/xfails.xfail
>
> Thanks, I believe
> https://gcc.gnu.org/pipermail/gcc-patches/2024-September/663711.html
> should fix this.

IIUC, you committed that followup patch as r15-3858-g2407dbe1d88706,
but I'm still seeing these errors in a more recent CI build  on
aarch64 (r15-3959-g87905f63a6521e)

It seems something in your followup patch prevented it from being
recognized by patchwork (I can't find it in
https://patchwork.sourceware.org/project/gcc/list/, and thus it didn't
trigger precommit validation where we could have easily checked that
it actually fixed the problem).

Thanks,

Christophe

>
> _______________________________________________
> linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org
> To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org
_______________________________________________
linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org
To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org

Reply via email to