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-28 Thread Manos Anagnostakis 
Sorry, forgot cc.

On Sat, Aug 19, 2023 at 10:24 PM Manos Anagnostakis <
manos.anagnosta...@vrull.eu> wrote:

> Hello Maxim,
>
> I've found out why this happens. It is a test problem having to do with
> -mcpu and should be trivial to address. I should have followed up with a V2
> by next week.
>
> Thank you for your report!
>
> Manos.
>
> On Sat, Aug 19, 2023 at 9:54 AM Maxim Kuvyrkov 
> wrote:
>
>> 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/
>>
>>
>>
>
> --
> *Manos Anagnostakis | Compiler Engineer |*
> E: manos.anagnosta...@vrull.eu 
>
> *VRULL GmbH *| Beatrixgasse 32 1030 Vienna |
>  W: www.vrull.eu | LinkedIn 
>


-- 
*Manos Anagnostakis | Compiler Engineer |*
E: manos.anagnosta...@vrull.eu 

*VRULL GmbH *| Beatrixgasse 32 1030 Vienna |
 W: www.vrull.eu | LinkedIn 
___
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-28 Thread Julian Brown 
On Sat, 19 Aug 2023 10:47:45 +0400
Maxim Kuvyrkov  wrote:

> 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.

Hi Maxim!

I'm a little confused -- to be clear, those patches aren't committed to
mainline yet, are they? If not, thanks for the proactive testing! If
so, oops, they've not been reviewed yet (!).

I'll take a look.

Cheers,

Julian
___
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: 2 regressions after gcc commit 27de9aa152141e7f3ee66372647d0f2cd94c4b90

2023-08-28 Thread Adhemerval Zanella Netto 
Hi Richard,

Your patch caused a regression [1] on aarch64-linux-gnu.  Would you
please investigate? I did a quick analysis and it seems that for
test_copy_lane_f32, test_copy_lane_s32, test_copy_lane_u32, gcc
is now generating zip1 instead of a ins; which does not seem
fully correct.

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

Thanks!

[1] 
https://ci.linaro.org/job/tcwg_gnu_cross_check_gcc--master-aarch64-build/912/artifact/artifacts/notify/mail-body.txt/*view*/
___
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: 2 regressions after gcc commit 27de9aa152141e7f3ee66372647d0f2cd94c4b90

2023-08-28 Thread Richard Biener 
On Mon, 28 Aug 2023, Adhemerval Zanella Netto wrote:

> Hi Richard,
> 
> Your patch caused a regression [1] on aarch64-linux-gnu.  Would you
> please investigate? I did a quick analysis and it seems that for
> test_copy_lane_f32, test_copy_lane_s32, test_copy_lane_u32, gcc
> is now generating zip1 instead of a ins; which does not seem
> fully correct.
> 
> Let me know if you need any assistance in reproducing these.

The testsuite likely needs adjustments or in case the old code
is better for some reason aarch64 vec_perm_const handling needs to
special case the insert permutes.

> Thanks!
> 
> [1] 
> https://ci.linaro.org/job/tcwg_gnu_cross_check_gcc--master-aarch64-build/912/artifact/artifacts/notify/mail-body.txt/*view*/
> 

-- 
Richard Biener 
SUSE Software Solutions Germany GmbH,
Frankenstrasse 146, 90461 Nuernberg, Germany;
GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)
___
linaro-toolchain mailing list -- linaro-toolchain@lists.linaro.org
To unsubscribe send an email to linaro-toolchain-le...@lists.linaro.org