[PATCH] D43117: [zorg] Cleanup unnecessary options for ARM and AArch64 bots

[Renato Golin via Phabricator via llvm-commits]

rengolin added inline comments.


================
Comment at: buildbot/osuosl/master/config/builders.py:248
                                         "-DLLVM_TARGETS_TO_BUILD='ARM;AArch64'",
-                                        "-DLLVM_LIT_ARGS='-sv -j2'",
                                         "-DLLVM_PARALLEL_LINK_JOBS=2"])},
 
----------------
You want to keep the -sv as that should very important information about the tests that failed.


================
Comment at: buildbot/osuosl/master/config/builders.py:287
                       runTestSuite=True,
                       testsuite_flags=['--cppflags', '-mcpu=cortex-a57',
                                        '--threads=8', '--build-threads=8'],
----------------
You're not changing it for LNT, better to do that for both.

I'd also try mcup=native instead...


================
Comment at: buildbot/osuosl/master/config/builders.py:326
                       runTestSuite=True,
-                      testsuite_flags=['--cppflags',
-                                       '-mcpu=cortex-a57 -O0 -mllvm -global-isel -mllvm -global-isel-abort=0',
+                      testsuite_flags=['--cppflags', '-mcpu=cortex-a57 -O0 -mllvm -global-isel -mllvm -global-isel-abort=0',
                                        '--threads=8', '--build-threads=8'],
----------------
@rovka is this still necessary? Isn't that the default for O0?

Also, mcpu changes. All or nothing.


================
Comment at: buildbot/osuosl/master/config/builders.py:998
                         checkout_lld=False,
-                        extra_cmake_args=["-DCMAKE_C_FLAGS='-mcpu=cortex-a15 -mfpu=vfpv3 -marm'",
-                                          "-DCMAKE_CXX_FLAGS='-mcpu=cortex-a15 -mfpu=vfpv3 -marm'",
-                                          "-DCOMPILER_RT_TEST_COMPILER_CFLAGS='-mcpu=cortex-a15 -mfpu=vfpv3 -marm'",
+                        extra_cmake_args=["-DCOMPILER_RT_TEST_COMPILER_CFLAGS='-mcpu=cortex-a15 -mfpu=vfpv3 -marm'",
                                           "-DLLVM_TARGETS_TO_BUILD='ARM;AArch64'",
----------------
Are the others being set correctly? They weren't when I did this.


================
Comment at: buildbot/osuosl/master/config/builders.py:1290
          'factory': LibcxxAndAbiBuilder.getLibcxxAndAbiBuilder(
             cmake_extra_opts={'LIBCXXABI_USE_LLVM_UNWINDER': 'ON',
                               'LLVM_PARALLEL_LINK_JOBS': '2'})},
----------------
why disabling marm/mthumb here?


Repository:
  rL LLVM

https://reviews.llvm.org/D43117