Add option to force filetype=asm for testing

pydir/run-pnacl-sz.py

 #!/usr/bin/env python2
 
 import argparse
 import itertools
 import os
 import re
 import subprocess
 import sys
 import tempfile
 
@@ skipping 60 matching lines @@
                            help='Assemble the output')
     argparser.add_argument('--disassemble', required=False,
                            action='store_true',
                            help='Disassemble the assembled output')
     argparser.add_argument('--dis-flags', required=False,
                            action='append', default=[],
                            help='Add a disassembler flag')
     argparser.add_argument('--filetype', default='iasm', dest='filetype',
                            choices=['obj', 'asm', 'iasm'],
                            help='Output file type.  Default %(default)s.')
+    argparser.add_argument('--forceasm', required=False, action='store_true',
+                           help='Force --filetype=asm.')

[Jim Stichnoth, 2015/12/20 18:42:23]

OCD: Remove the '.' from the help string for consistency.

And in the same vein:
  s/.  Default %(default)s.//
because formatter_class=argparse.ArgumentDefaultsHelpFormatter) makes that
unnecessary.

[sehr, 2016/01/07 18:53:11]

Done.

     argparser.add_argument('--target', default='x8632', dest='target',
                            choices=['x8632','arm32','mips32'],
                            help='Target architecture.  Default %(default)s.')
     argparser.add_argument('--echo-cmd', required=False,
                            action='store_true',
                            help='Trace command that generates ICE instructions')
     argparser.add_argument('--tbc', required=False, action='store_true',
                            help='Input is textual bitcode (not .ll)')
     argparser.add_argument('--expect-fail', required=False, action='store_true',
                            help='Negate success of run by using LLVM not')
@@ skipping 13 matching lines @@
     if args.llvm_source and args.no_local_syms:
       raise RuntimeError("Can't specify both '--llvm-source' and " +
                          "'--no-local-syms'")
 
     if args.llvm_source and args.tbc:
       raise RuntimeError("Can't specify both '--tbc' and '--llvm-source'")
 
     if args.llvm and args.tbc:
       raise RuntimeError("Can't specify both '--tbc' and '--llvm'")
 
+    if args.forceasm:
+      if args.filetype == 'asm':
+        pass
+      elif args.filetype == 'iasm':
+        # TODO(sehr) implement forceasm for iasm.
+        pass
+      elif args.filetype == 'obj':
+        args.filetype = 'asm'
+        args.assemble = True
+
     cmd = []
     if args.tbc:
       cmd = [os.path.join(pnacl_bin_path, 'pnacl-bcfuzz'), llfile,
              '-bitcode-as-text', '-output', '-', '|']
     elif not args.llvm_source:
       cmd = [os.path.join(pnacl_bin_path, 'llvm-as'), llfile, '-o', '-', '|',
              os.path.join(pnacl_bin_path, 'pnacl-freeze')]
       if not args.no_local_syms:
         cmd += ['--allow-local-symbol-tables']
       cmd += ['|']
@@ skipping 41 matching lines @@
               [asm_temp.name])
 
     stdout_result = shellcmd(cmd, echo=args.echo_cmd)
     if not args.echo_cmd:
       sys.stdout.write(stdout_result)
     if asm_temp:
       os.remove(asm_temp.name)
 
 if __name__ == '__main__':
     main()