Re: [Linaro-TCWG-CI] FAIL: 6 regressions after gcc commit: basepoints/gcc-14-3331-gcddc26e0274 aarch64: Fine-grained ldp and stp policies with test-cases.

2023-08-18 Thread Maxim Kuvyrkov
Hi Manos,

New tests in your patch [1] fail on aarch64-linux-gnu build in our CI.  Would 
you please investigate why?  Testing logs are at [2].

[1] 
https://patchwork.sourceware.org/project/gcc/patch/20230818074943.41754-1-manos.anagnosta...@vrull.eu/
[2] 
https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-precommit/1602/artifact/artifacts/artifacts.precommit/00-sumfiles/

--
Maxim Kuvyrkov
https://www.linaro.org

> On Aug 19, 2023, at 08:37, ci_not...@linaro.org wrote:
> 
> [Linaro-TCWG-CI] FAIL: 6 regressions after gcc commit: 
> basepoints/gcc-14-3331-gcddc26e0274 aarch64: Fine-grained ldp and stp 
> policies with test-cases.
> 
> Dear contributor, our automatic CI has detected problems related to your 
> patch.
> Please find below some details about it.
> 
> In CI config tcwg_gcc_check/master-aarch64 after:
> 
>  | commit cddc26e0274b51e775929e497f89d203211689d2
>  | Author: Manos Anagnostakis 
>  | Date:   Fri Aug 18 10:49:43 2023 +0300
>  | 
>  | aarch64: Fine-grained ldp and stp policies with test-cases.
>  | 
>  | This patch implements the following TODO in 
> gcc/config/aarch64/aarch64.cc
>  | to provide the requested behaviour for handling ldp and stp:
>  | 
>  |   /* Allow the tuning structure to disable LDP instruction formation
>  | ... 47 lines of the commit log omitted.
> 
> FAIL: 6 regressions
> 
> regressions.sum:
> === gcc tests ===
> 
> Running gcc:gcc.target/aarch64/aarch64.exp ...
> FAIL: gcc.target/aarch64/ldp_aligned.c scan-assembler-times ldp\tq[0-9]+, 
> q[0-9] 1
> FAIL: gcc.target/aarch64/ldp_aligned.c scan-assembler-times ldp\tw[0-9]+, 
> w[0-9] 3
> FAIL: gcc.target/aarch64/ldp_aligned.c scan-assembler-times ldp\tx[0-9]+, 
> x[0-9] 3
> FAIL: gcc.target/aarch64/ldp_always.c scan-assembler-times ldp\tq[0-9]+, 
> q[0-9] 2
> FAIL: gcc.target/aarch64/ldp_always.c scan-assembler-times ldp\tw[0-9]+, 
> w[0-9] 6
> FAIL: gcc.target/aarch64/ldp_always.c scan-assembler-times ldp\tx[0-9]+, 
> x[0-9] 6
> 
> ... and 1 more entries
> 
> 
> 
> -8<--8<--8<--
> The information below can be used to reproduce a debug environment:
> 
> Current build   : 
> https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-precommit/1602/artifact/artifacts
> Reference build : 
> https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/836/artifact/artifacts
> 
> Reproduce last good and first bad builds: 
> https://git.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sha1/cddc26e0274b51e775929e497f89d203211689d2/reproduction_instructions.txt
> 
> Full commit : 
> https://github.com/gcc-mirror/gcc/commit/cddc26e0274b51e775929e497f89d203211689d2
> 
> Latest bug report status : https://linaro.atlassian.net/browse/GNU-692
> 
> List of configurations that regressed due to this commit :
> * tcwg_gcc_check
> ** master-aarch64
> *** FAIL: 6 regressions
> *** https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-precommit/1602/


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


Re: [Linaro-TCWG-CI] FAIL: 10 regressions after gcc commit: 5 commits in gcc

2023-08-18 Thread Maxim Kuvyrkov
Hi Julian,

Your patch series causes regressions on aarch64-linux-gnu.  Would you please 
investigate?

Let me know if you need any assistance in reproducing these.

Thanks!

--
Maxim Kuvyrkov
https://www.linaro.org

> On Aug 19, 2023, at 09:32, ci_not...@linaro.org wrote:
> 
> [Linaro-TCWG-CI] FAIL: 10 regressions after gcc commit: 5 commits in gcc
> 
> Dear contributor, our automatic CI has detected problems related to your 
> patch.
> Please find below some details about it.
> 
> In CI config tcwg_gcc_check/master-aarch64 after:
> 
>  | gcc commits:
>  | dce6c135fb52fd631c2fc82d8048d32ce41ece21 OpenMP/OpenACC: Reorganise OMP 
> map clause handling in gimplify.cc
>  | dd49dd178e3eac8e9925baa3d71325d8d5f69215 OpenMP/OpenACC: 
> Unordered/non-constant component offset runtime diagnostic
>  | bd5a53e6b47907d05672cbb603af363a665b45a4 OpenMP: Pointers and member 
> mappings
>  | 4e0359d8a659c8abdca3297fc9b0e20ff89f7f82 OpenMP/OpenACC: Rework clause 
> expansion and nested struct handling
>  | a855174e5461d2b423af7f892fd31dfb10ce09ec OpenMP/OpenACC: Reindent 
> TO/FROM/_CACHE_ stanza in {c_}finish_omp_clause
> 
> FAIL: 10 regressions
> 
> regressions.sum:
> === libgomp tests ===
> 
> Running libgomp:libgomp.c++/c++.exp ...
> FAIL: libgomp.c++/../libgomp.c-c++-common/map-arrayofstruct-2.c output 
> pattern test
> FAIL: libgomp.c++/../libgomp.c-c++-common/map-arrayofstruct-3.c output 
> pattern test
> 
> Running libgomp:libgomp.c/c.exp ...
> FAIL: libgomp.c/../libgomp.c-c++-common/map-arrayofstruct-2.c output pattern 
> test
> FAIL: libgomp.c/../libgomp.c-c++-common/map-arrayofstruct-3.c output pattern 
> test
> 
> ... and 9 more entries
> 
> 
> 
> -8<--8<--8<--
> The information below can be used to reproduce a debug environment:
> 
> Current build   : 
> https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-precommit/1593/artifact/artifacts
> Reference build : 
> https://ci.linaro.org/job/tcwg_gcc_check--master-aarch64-build/836/artifact/artifacts


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


[ACTIVITY] Report for week #33

2023-08-18 Thread Thiago Jung Bauermann

Hello,

# [GNU-767] Support changing SVE vector length in remote debugging

- Continued working on new approach to support changing SVE vector
  length in remote debugging.

# TCWG CI

- Analysed GCC regression GNU-880 which is actually an XFAIL → XPASS so
  I sent a patch removing the xfail annotation.

- Analysed GCC regression GNU-881. The problem is that we don't have
  Python in the ABE sysroot so a new Python GCC plugin testcase fails to
  build. Sent a patch to detect that situation and mark the test as
  unsupported. It was committed upstream.

- Tested a couple of new versions of a GDB mailing list patch to see if
  it fixed the failure reported by the precommit CI. Reported results to
  patch author.

- Fixed silly mistake spotted by Laurent in ABE Gerrit requests to use
  TIMEOUTFACTOR for DejaGnu testsuites. Adjusted it to not use the
  factor in GDB on armhf and sent new version for review.

- Reviewed a couple of Gerrit requests.

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


[ACTIVITY] report week ending 18 Aug

2023-08-18 Thread Peter Maydell
Progress (short week, 3 days):
 * UM-2 [QEMU upstream maintainership]
  - sent out another couple of "avoid VLA" patches
 * QEMU-530 [QEMU ARM v9.4 Baseline CPU for TCG]
  - managed to write some first-pass code for the MTE checks for
FEAT_MOPS memset operations. Got something together enough to
send to RTH for some pre-review before I roll the approach out
to the other insns.

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


Re: [Linaro-TCWG-CI] Failure after gcc commit: basepoints/gcc-14-3309-g360cabf45a0 arm: Add define_attr to to create a mapping between MVE predicated and unpredicated insns

2023-08-18 Thread Stam Markianos-Wright
Ack, indeed, sorry about that. I had forgotten that there was a dependency to 
this https://gcc.gnu.org/pipermail/gcc-patches/2023-August/625977.html since 
rebasing the patch series. I'll ping that and get it committed





From: Maxim Kuvyrkov 
Sent: Friday, August 18, 2023 7:58 AM
To: Stam Markianos-Wright 
Cc: Linaro Toolchain Working Group ; Thiago 
Bauermann 
Subject: Re: [Linaro-TCWG-CI] Failure after gcc commit: 
basepoints/gcc-14-3309-g360cabf45a0 arm: Add define_attr to to create a mapping 
between MVE predicated and unpredicated insns

Warning: EXTERNAL SENDER, use caution when opening links or attachments.


Hi Stamatis,

Let us know if you need any help in troubleshooting the failures in 
https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchwork.sourceware.org%2Fproject%2Fgcc%2Flist%2F%3Fseries%3D23558&data=05%7C01%7Cstam.markianos-wright%40arm.com%7Cf9ece4881c7e432ba0da08db9fb885d0%7Cf34e597957d94d4db122a662184d%7C0%7C0%7C638279387096463608%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=H9ZqaaojdQ2B2%2Baz0PlmZb95lee30MdjTTfpAtYvEsw%3D&reserved=0
 .

Kind regards,

--
Maxim Kuvyrkov
https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.linaro.org%2F&data=05%7C01%7Cstam.markianos-wright%40arm.com%7Cf9ece4881c7e432ba0da08db9fb885d0%7Cf34e597957d94d4db122a662184d%7C0%7C0%7C638279387096463608%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Ibitl0DqQuauJQtiHzP4kiSzoPafdNYHGsD%2BGF9ilFo%3D&reserved=0


> On Aug 18, 2023, at 01:13, ci_not...@linaro.org wrote:
>
> [Linaro-TCWG-CI] Failure after gcc commit: 
> basepoints/gcc-14-3309-g360cabf45a0 arm: Add define_attr to to create a 
> mapping between MVE predicated and unpredicated insns
>
> Dear contributor, our automatic CI has detected problems related to your 
> patch.
> Please find below some details about it.
>
> In CI config tcwg_gcc_build/master-arm after:
>
>  | commit 360cabf45a05d5d2d5a98f0e081cb8f53e01e8aa
>  | Author: Stamatis Markianos-Wright 
>  | Date:   Thu Aug 17 11:30:58 2023 +0100
>  |
>  | arm: Add define_attr to to create a mapping between MVE predicated and 
> unpredicated insns
>  |
>  | Hi all,
>  |
>  | I'd like to submit two patches that add support for Arm's MVE
>  | Tail Predicated Low Overhead Loop feature.
>
> Results changed to
> # reset_artifacts:
> -10
> # true:
> 0
> # build_abe gcc:
> # FAILED
> # First few build errors in logs:
> # 00:02:46 
> /home/tcwg-build/workspace/tcwg_gnu_1/abe/snapshots/gcc.git~master/gcc/config/arm/mve.md:7018:8:
>  error: invalid decimal constant
> # 00:02:46 make[2]: *** [Makefile:2647: s-preds-h] Error 1
> # 00:02:46 make[1]: *** [Makefile:4655: all-gcc] Error 2
> # 00:02:46 make: *** [Makefile:1051: all] Error 2
> # 00:02:28 
> /home/tcwg-build/workspace/tcwg_gnu_1/abe/snapshots/gcc.git~master/gcc/config/arm/mve.md:7018:8:
>  error: invalid decimal constant
> # 00:02:28 make[2]: *** [Makefile:2586: s-conditions] Error 1
> # 00:02:28 
> /home/tcwg-build/workspace/tcwg_gnu_1/abe/snapshots/gcc.git~master/gcc/config/arm/mve.md:7018:8:
>  error: invalid decimal constant
> # 00:02:28 
> /home/tcwg-build/workspace/tcwg_gnu_1/abe/snapshots/gcc.git~master/gcc/config/arm/mve.md:7018:8:
>  error: invalid decimal constant
> # 00:02:28 make[2]: *** [Makefile:2647: s-preds-h] Error 1
> # 00:02:28 make[2]: *** [Makefile:2652: s-constrs-h] Error 1
> # 00:02:28 
> /home/tcwg-build/workspace/tcwg_gnu_1/abe/snapshots/gcc.git~master/gcc/config/arm/mve.md:7018:8:
>  error: invalid decimal constant
> # 00:02:28 make[2]: *** [Makefile:2642: s-preds] Error 1
> # 00:02:35 make[1]: *** [Makefile:4655: all-gcc] Error 2
> # 00:02:35 make: *** [Makefile:1051: all] Error 2
>
> From
> # reset_artifacts:
> -10
> # true:
> 0
> # build_abe gcc:
> 1
>
>
>
> -8<--8<--8<--
> The information below can be used to reproduce a debug environment:
>
> Current build   : 
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fci.linaro.org%2Fjob%2Ftcwg_gcc_build--master-arm-precommit%2F1787%2Fartifact%2Fartifacts&data=05%7C01%7Cstam.markianos-wright%40arm.com%7Cf9ece4881c7e432ba0da08db9fb885d0%7Cf34e597957d94d4db122a662184d%7C0%7C0%7C638279387096463608%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=7TsPmqjCXaZwu%2FCeuDN1bjFE67MotjNRVPl0ZecFwTU%3D&reserved=0
> Reference build : 
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fci.linaro.org%2Fjob%2Ftcwg_gcc_build--master-arm-build%2F1060%2Fartifact%2Fartifacts&data=05%7C01%7Cstam.markianos-wright%40arm.co