Re: [Linaro-TCWG-CI] llvmorg-18-init-7933-ge13bed4c5f35: slowed down by 6% - 464.h264ref on aarch64 O2

2023-10-09 Thread Dmitriy Smirnov
Hi Maxim,

Could you put a comment about the issue on Phabricator please 
(https://reviews.llvm.org/D155688). Also, is the reason for the regressions 
already known?

Thank you.

--
Dmitriy

From: Maxim Kuvyrkov 
Sent: 09 October 2023 10:58
To: Dmitriy Smirnov 
Cc: Paul Walker ; Linaro Toolchain Working Group 

Subject: Fwd: [Linaro-TCWG-CI] llvmorg-18-init-7933-ge13bed4c5f35: slowed down 
by 6% - 464.h264ref on aarch64 O2

Hi Dmitriy,

Linaro Benchmarking CI has flagged several interesting code-speed and code-size 
regressions for your patch -- see [1].

In particular, could you check if below regressions can be avoided:
- grew in size by 21% - 473.astar:[.] _ZN7way2obj12releasepointEii
- slowed down by 61% - 505.mcf_r:[.] price_out_impl

Both of these are for 32-bit ARM, but AArch64 also has code-speed and code-size 
regressions.

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

[1] https://linaro.atlassian.net/browse/LLVM-1001

Kind regards,

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

> Begin forwarded message:
>
> From: ci_not...@linaro.org
> Subject: [Linaro-TCWG-CI] llvmorg-18-init-7933-ge13bed4c5f35: slowed down by 
> 6% - 464.h264ref on aarch64 O2
> Date: October 8, 2023 at 04:26:39 GMT+4
> To: maxim.kuvyr...@linaro.org
> Reply-To: linaro-toolchain@lists.linaro.org
>
> 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.
>
> In CI config tcwg_bmk-code_speed-spec2k6/llvm-aarch64-master-O2 after:
>
>  | commit llvmorg-18-init-7933-ge13bed4c5f35
>  | Author: Dmitriy Smirnov 
>  | Date:   Fri Oct 6 11:15:00 2023 +0100
>  |
>  | [PATCH] [llvm] [InstCombine] Canonicalise ADD+GEP
>  |
>  | This patch tries to canonicalise add + gep to gep + gep.
>  |
>  | Co-authored-by: Paul Walker 
>  |
>  | Reviewed By: nikic
>  | ... 2 lines of the commit log omitted.
>
> the following benchmarks slowed down by more than 3%:
> - slowed down by 6% - 464.h264ref - from 11126 to 11766 perf samples
> the following hot functions slowed down by more than 15% (but their 
> benchmarks slowed down by less than 3%):
> - slowed down by 44% - 464.h264ref:[.] FastFullPelBlockMotionSearch - from 
> 1531 to 2206 perf samples
>
> The configuration of this build is:
> Below reproducer instructions can be used to re-build both "first_bad" and 
> "last_good" cross-toolchains used in this bisection.  Naturally, the scripts 
> will fail when triggerring benchmarking jobs if you don\'t have access to 
> Linaro TCWG CI.
>
> Configuration:
> - Benchmark:
> - Toolchain: Clang + Glibc + LLVM Linker
> - Version: all components were built from their tip of trunk
> - Target: aarch64-linux-gnu
> - Compiler flags: O2
> - Hardware: NVidia TX1 4x Cortex-A57
>
> This benchmarking CI is work-in-progress, and we welcome feedback and 
> suggestions at linaro-toolchain@lists.linaro.org .  In our improvement plans 
> is to add support for SPEC CPU2017 benchmarks and provide "perf 
> report/annotate" data behind these reports.
>
> -8<--8<--8<--
> The information below can be used to reproduce a debug environment:
>
> Current build   : 
> https://ci.linaro.org/job/tcwg_bmk-code_speed-spec2k6--llvm-aarch64-master-O2-build/142/artifact/artifacts
> Reference build : 
> https://ci.linaro.org/job/tcwg_bmk-code_speed-spec2k6--llvm-aarch64-master-O2-build/141/artifact/artifacts
>
> Reproduce last good and first bad builds: 
> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/llvm/sha1/e13bed4c5f3544c076ce57e36d9a11eefa5a7815/tcwg_bmk-code_speed-spec2k6/llvm-aarch64-master-O2/reproduction_instructions.txt
>
> Full commit : 
> https://github.com/llvm/llvm-project/commit/e13bed4c5f3544c076ce57e36d9a11eefa5a7815
>
> Latest bug report status : https://linaro.atlassian.net/browse/LLVM-1001
>
> List of configurations that regressed due to this commit :
> * tcwg_bmk-code_speed-spec2k6
> ** llvm-aarch64-master-O2
> *** slowed down by 6% - 464.h264ref
> *** 
> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/llvm/sha1/e13bed4c5f3544c076ce57e36d9a11eefa5a7815/tcwg_bmk-code_speed-spec2k6/llvm-aarch64-master-O2/details.txt
> *** 
> https://ci.linaro.org/job/tcwg_bmk-code_speed-spec2k6--llvm-aarch64-master-O2-build/142/


IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
___
linaro-toolchain ma

Fwd: [Linaro-TCWG-CI] llvmorg-18-init-7933-ge13bed4c5f35: slowed down by 6% - 464.h264ref on aarch64 O2

2023-10-09 Thread Maxim Kuvyrkov
Hi Dmitriy,

Linaro Benchmarking CI has flagged several interesting code-speed and code-size 
regressions for your patch -- see [1].

In particular, could you check if below regressions can be avoided:
- grew in size by 21% - 473.astar:[.] _ZN7way2obj12releasepointEii
- slowed down by 61% - 505.mcf_r:[.] price_out_impl

Both of these are for 32-bit ARM, but AArch64 also has code-speed and code-size 
regressions.

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

[1] https://linaro.atlassian.net/browse/LLVM-1001

Kind regards,

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

> Begin forwarded message:
> 
> From: ci_not...@linaro.org
> Subject: [Linaro-TCWG-CI] llvmorg-18-init-7933-ge13bed4c5f35: slowed down by 
> 6% - 464.h264ref on aarch64 O2
> Date: October 8, 2023 at 04:26:39 GMT+4
> To: maxim.kuvyr...@linaro.org
> Reply-To: linaro-toolchain@lists.linaro.org
> 
> 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.
> 
> In CI config tcwg_bmk-code_speed-spec2k6/llvm-aarch64-master-O2 after:
> 
>  | commit llvmorg-18-init-7933-ge13bed4c5f35
>  | Author: Dmitriy Smirnov 
>  | Date:   Fri Oct 6 11:15:00 2023 +0100
>  | 
>  | [PATCH] [llvm] [InstCombine] Canonicalise ADD+GEP
>  | 
>  | This patch tries to canonicalise add + gep to gep + gep.
>  | 
>  | Co-authored-by: Paul Walker 
>  | 
>  | Reviewed By: nikic
>  | ... 2 lines of the commit log omitted.
> 
> the following benchmarks slowed down by more than 3%:
> - slowed down by 6% - 464.h264ref - from 11126 to 11766 perf samples
> the following hot functions slowed down by more than 15% (but their 
> benchmarks slowed down by less than 3%):
> - slowed down by 44% - 464.h264ref:[.] FastFullPelBlockMotionSearch - from 
> 1531 to 2206 perf samples
> 
> The configuration of this build is:
> Below reproducer instructions can be used to re-build both "first_bad" and 
> "last_good" cross-toolchains used in this bisection.  Naturally, the scripts 
> will fail when triggerring benchmarking jobs if you don\'t have access to 
> Linaro TCWG CI.
> 
> Configuration:
> - Benchmark: 
> - Toolchain: Clang + Glibc + LLVM Linker
> - Version: all components were built from their tip of trunk
> - Target: aarch64-linux-gnu
> - Compiler flags: O2
> - Hardware: NVidia TX1 4x Cortex-A57
> 
> This benchmarking CI is work-in-progress, and we welcome feedback and 
> suggestions at linaro-toolchain@lists.linaro.org .  In our improvement plans 
> is to add support for SPEC CPU2017 benchmarks and provide "perf 
> report/annotate" data behind these reports.
> 
> -8<--8<--8<--
> The information below can be used to reproduce a debug environment:
> 
> Current build   : 
> https://ci.linaro.org/job/tcwg_bmk-code_speed-spec2k6--llvm-aarch64-master-O2-build/142/artifact/artifacts
> Reference build : 
> https://ci.linaro.org/job/tcwg_bmk-code_speed-spec2k6--llvm-aarch64-master-O2-build/141/artifact/artifacts
> 
> Reproduce last good and first bad builds: 
> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/llvm/sha1/e13bed4c5f3544c076ce57e36d9a11eefa5a7815/tcwg_bmk-code_speed-spec2k6/llvm-aarch64-master-O2/reproduction_instructions.txt
> 
> Full commit : 
> https://github.com/llvm/llvm-project/commit/e13bed4c5f3544c076ce57e36d9a11eefa5a7815
> 
> Latest bug report status : https://linaro.atlassian.net/browse/LLVM-1001
> 
> List of configurations that regressed due to this commit :
> * tcwg_bmk-code_speed-spec2k6
> ** llvm-aarch64-master-O2
> *** slowed down by 6% - 464.h264ref
> *** 
> https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/llvm/sha1/e13bed4c5f3544c076ce57e36d9a11eefa5a7815/tcwg_bmk-code_speed-spec2k6/llvm-aarch64-master-O2/details.txt
> *** 
> https://ci.linaro.org/job/tcwg_bmk-code_speed-spec2k6--llvm-aarch64-master-O2-build/142/


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