Seems... unlikely this change had a performance impact.

Also is this email meant to be sent to public contributors like myself, or
only intended for some Linaro folks?

On Sun, Dec 5, 2021 at 6:18 AM <ci_not...@linaro.org> wrote:

> After llvm commit bd4c6a476fd037fb07a1c484f75d93ee40713d3d
> Author: David Blaikie <dblai...@gmail.com>
>
>     Add missing header
>
> the following benchmarks slowed down by more than 2%:
> - 433.milc slowed down by 4% from 12427 to 12916 perf samples
>
> 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.
>
> For your convenience, we have uploaded tarballs with pre-processed source
> and assembly files at:
> - First_bad save-temps:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/build-bd4c6a476fd037fb07a1c484f75d93ee40713d3d/save-temps/
> - Last_good save-temps:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/build-7d4da4e1ab7f79e51db0d5c2a0f5ef1711122dd7/save-temps/
> - Baseline save-temps:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/build-baseline/save-temps/
>
> Configuration:
> - Benchmark: SPEC CPU2006
> - Toolchain: Clang + Glibc + LLVM Linker
> - Version: all components were built from their tip of trunk
> - Target: aarch64-linux-gnu
> - Compiler flags: -O2 -flto
> - 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.
>
> THIS IS THE END OF INTERESTING STUFF.  BELOW ARE LINKS TO BUILDS,
> REPRODUCTION INSTRUCTIONS, AND THE RAW COMMIT.
>
> This commit has regressed these CI configurations:
>  - tcwg_bmk_llvm_tx1/llvm-master-aarch64-spec2k6-O2_LTO
>
> First_bad build:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/build-bd4c6a476fd037fb07a1c484f75d93ee40713d3d/
> Last_good build:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/build-7d4da4e1ab7f79e51db0d5c2a0f5ef1711122dd7/
> Baseline build:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/build-baseline/
> Even more details:
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/
>
> Reproduce builds:
> <cut>
> mkdir investigate-llvm-bd4c6a476fd037fb07a1c484f75d93ee40713d3d
> cd investigate-llvm-bd4c6a476fd037fb07a1c484f75d93ee40713d3d
>
> # Fetch scripts
> git clone https://git.linaro.org/toolchain/jenkins-scripts
>
> # Fetch manifests and test.sh script
> mkdir -p artifacts/manifests
> curl -o artifacts/manifests/build-baseline.sh
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/manifests/build-baseline.sh
> --fail
> curl -o artifacts/manifests/build-parameters.sh
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/manifests/build-parameters.sh
> --fail
> curl -o artifacts/test.sh
> https://ci.linaro.org/job/tcwg_bmk_ci_llvm-bisect-tcwg_bmk_tx1-llvm-master-aarch64-spec2k6-O2_LTO/34/artifact/artifacts/test.sh
> --fail
> chmod +x artifacts/test.sh
>
> # Reproduce the baseline build (build all pre-requisites)
> ./jenkins-scripts/tcwg_bmk-build.sh @@
> artifacts/manifests/build-baseline.sh
>
> # Save baseline build state (which is then restored in artifacts/test.sh)
> mkdir -p ./bisect
> rsync -a --del --delete-excluded --exclude /bisect/ --exclude /artifacts/
> --exclude /llvm/ ./ ./bisect/baseline/
>
> cd llvm
>
> # Reproduce first_bad build
> git checkout --detach bd4c6a476fd037fb07a1c484f75d93ee40713d3d
> ../artifacts/test.sh
>
> # Reproduce last_good build
> git checkout --detach 7d4da4e1ab7f79e51db0d5c2a0f5ef1711122dd7
> ../artifacts/test.sh
>
> cd ..
> </cut>
>
> Full commit (up to 1000 lines):
> <cut>
> commit bd4c6a476fd037fb07a1c484f75d93ee40713d3d
> Author: David Blaikie <dblai...@gmail.com>
> Date:   Mon Nov 29 16:29:25 2021 -0800
>
>     Add missing header
> ---
>  llvm/lib/Demangle/DLangDemangle.cpp | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/llvm/lib/Demangle/DLangDemangle.cpp
> b/llvm/lib/Demangle/DLangDemangle.cpp
> index faf91b239490..f380aa90035e 100644
> --- a/llvm/lib/Demangle/DLangDemangle.cpp
> +++ b/llvm/lib/Demangle/DLangDemangle.cpp
> @@ -17,6 +17,7 @@
>  #include "llvm/Demangle/StringView.h"
>  #include "llvm/Demangle/Utility.h"
>
> +#include <cctype>
>  #include <cstring>
>  #include <limits>
>
> </cut>
_______________________________________________
linaro-toolchain mailing list
linaro-toolchain@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/linaro-toolchain

Reply via email to