[SwiftSyntax] Clean build script

swiftlang/swift-syntax#458

Author: kimdv

utils/swift_build_support/swift_build_support/products/swiftsyntax.py

@@ -49,27 +49,23 @@ def is_before_build_script_impl_product(cls):
     def is_swiftpm_unified_build_product(cls):
         return True
 
-    def run_swiftsyntax_build_script(self, target, additional_params=[]):
+    def run_swiftsyntax_build_script(self, target, command, additional_params=[]):
         llvm_build_dir = os.path.join(self.build_dir, '..', 'llvm-' + target)
         llvm_build_dir = os.path.realpath(llvm_build_dir)
 
         script_path = os.path.join(self.source_dir, 'build-script.py')
 
         build_cmd = [
             script_path,
+            command,
             '--build-dir', self.build_dir,
             '--multiroot-data-file', MULTIROOT_DATA_FILE_PATH,
-            '--toolchain', self.install_toolchain_path(target),
-            '--filecheck-exec', os.path.join(llvm_build_dir, 'bin',
-                                             'FileCheck'),
+            '--toolchain', self.install_toolchain_path(target)
         ]
 
         if self.is_release():
             build_cmd.append('--release')
 
-        if self.args.swiftsyntax_verify_generated_files:
-            build_cmd.append('--verify-generated-files')
-
         build_cmd.extend(additional_params)
 
         if self.args.verbose_build:
@@ -81,14 +77,23 @@ def should_build(self, host_target):
         return True
 
     def build(self, host_target):
-        self.run_swiftsyntax_build_script(target=host_target)
+        if self.args.swiftsyntax_verify_generated_files:
+            self.run_swiftsyntax_build_script(target=host_target,
+                                                command='verify-source-code')
+
+        self.run_swiftsyntax_build_script(target=host_target, 
+                                          command='build')
 
     def should_test(self, host_target):
         return self.args.test_swiftsyntax
 
-    def test(self, host_target):
+    def test(self, host_target):        
         self.run_swiftsyntax_build_script(target=host_target,
-                                          additional_params=['--test'])
+                                          command='test',
+                                          additional_params=[
+                                            '--filecheck-exec', os.path.join(llvm_build_dir, 'bin',
+                                                'FileCheck')
+                                             ])
 
     def should_install(self, host_target):
         return self.args.install_swiftsyntax