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

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 9 06:51:36 PST 2018


rovka added inline comments.


================
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'],
----------------
rengolin wrote:
> @rovka is this still necessary? Isn't that the default for O0?
> 
> Also, mcpu changes. All or nothing.
Nope, we should only need -O0 now. 


Repository:
  rL LLVM

https://reviews.llvm.org/D43117





More information about the llvm-commits mailing list