I've updated buildbot.tac. Thanks, Galina.

@gkistanova
Hi Galina,

Galina, Diana, any other feedback? Or shall I commit the patch?

For reference, here are testing builds:

• http://ex40-01.tcwglab.linaro.org:8011/builders/clang-arm64-windows-msvc
• http://ex40-01.tcwglab.linaro.org:8011/builders/clang-arm64-windows-msvc-2stage

LGTM.

Hi @mkazantsev ,
Linaro benchmarking CI flagged this patch as increases code-size of SPEC2k6's 401.bzip2 by 3% on ARM (Thumb2 mode) and by 5% on AArch64. This happens at -Os -flto.

Hi Sam,
Linaro benchmarking CI flagged this patch as it regresses SPEC2k6's 462.libquantum by 10% at -O2, -O3 and "-O3 -flto" for aarch64-linux-gnu. Impact on other benchmarks is below 2%, which is within noise.

Quite surprisingly, this patch increases code size by 3-4% on several SPEC CPU2006 benchmarks for arm-linux-gnueabihf at -Oz optimization level:

1. 473.astar,astar_base.default regressed by 103
2. 450.soplex,soplex_base.default regressed by 103
3. 483.xalancbmk,Xalan_base.default regressed by 104

It's runtime failure and occurs on "ref" input. Compilation flags are
-O3 -marm --target=armv7a-linux-gnueabihf --sysroot=/path/to/build/sysroots/arm-linux-gnueabihf
and linking flags are (note that I'm linking with LLD).
-Wl,--build-id -Wl,-dynamic-linker=/path/to/run/sysroot/lib/ld-2.31.9000.so -Wl,-rpath=/path/to/run/sysroot/lib -Wl,-rpath=/path/to/run/sysroot/usr/lib -O3 -marm --target=armv7a-linux-gnueabihf --sysroot=/path/to/build/sysroots/arm-linux-gnueabihf -fuse-ld=lld

@MaskRay , hi. This patch miscompiles 471.omnetpp and 453.povray on arm-linux-gnueabihf. Would you please investigate?

This commit 4698bf145d583e26ed438026ef7fde031ef322b1

Author: Kazu Hirata <kazu@google.com>
Date: Wed Feb 5 08:24:01 2020 -0800

Hi Nikita,

Hi @kazu ,
This patch increases code-size of 444.namd by 1% on arm-linux-gnueabihf:

• when compiling with -Os: from 171653 to 173645
• when compiling with -Os -flto: from 137772 to 139708

Could you take a look if these regressions can be easily avoided?
Let me know if you need help reproducing the regressions. I have pre-processed source and generated assembly handy.
Thanks!

Hi John,

Looks good.

@leandron , note that you have to commit this patch yourself or (if you don't have commit access) explicitly ask someone here to commit it for you.

Gentle ping? This is a fairly simple patch. Without it I can't enable linaro-armv8-01-arm-libcxx* bots.

Correct patch. First upload had unrelated changes.

Rebase and add -mcpu=cortex-a57 to armv8 32-bit bots.

I'll amend libcxx and global-isel builders per comments in the final commit. Changes for all other bots seem to be OK.

This is being tested on my local buildmaster: http://ex40-01.tcwglab.linaro.org:32775/builders/clang-cmake-armv8-global-isel/builds/

This is a trivial cleanup that was split up from an approved patch: https://reviews.llvm.org/D42039

Rebase on trunk

@gkistanova , would you please merge this before your next planned update to buildmaster? Thanks!

Any yay/nay?

@rovka , slaves linaro-d05-01-* will run in different containers on the same machine (linaro-d05-01). In this context there is no need to repurpose a slave.
I plan to add a second machine linaro-d05-02 that would host the same set of slaves/builders for redundancy.