[PATCH] D27766: [buildbot] Added CUDATestsuiteBuilder.py

[Justin Lebar via Phabricator via llvm-commits]

jlebar added inline comments.


================
Comment at: zorg/buildbot/builders/CUDATestsuiteBuilder.py:86
+                    '-DTEST_SUITE_SUBDIRS=External',
+                    ] + extra_ts_cmake_args
+
----------------
Nit, this would be clearer to me if it was indented like cuda_test_env above.


================
Comment at: zorg/buildbot/builders/CUDATestsuiteBuilder.py:135
+    # WARNING: This takes a lot of time to build.
+    if (enable_thrust_tests):
+        f.addStep(WarningCountingShellCommand(
----------------
Nit, no parens in if.


https://reviews.llvm.org/D27766