Subzero. Misc ARM32 bugfixes.

pydir/szbuild.py

 #!/usr/bin/env python2
 
 import argparse
 import os
 import pipes
 import re
 import sys
 
 from utils import shellcmd
 from utils import FindBaseNaCl
@@ skipping 70 matching lines @@
                                 '(m1 and -1 are equivalent).' +
                                 ' Default %(default)s.')
     argparser.add_argument('--filetype', default='iasm', dest='filetype',
                            choices=['obj', 'asm', 'iasm'],
                            help='Output file type.  Default %(default)s.')
     argparser.add_argument('--sandbox', dest='sandbox', action='store_true',
                            help='Enable sandboxing in the translator')
     argparser.add_argument('--enable-block-profile',
                            dest='enable_block_profile', action='store_true',
                            help='Enable basic block profiling.')
+    argparser.add_argument('--target', default='x8632', dest='target',
+                           choices=['arm32', 'x8632'],
+                           help='Tells Subzero to generate code for the '

[Jim Stichnoth, 2015/10/15 23:34:21]

Bikeshed:
  help='Generate code for specified target'

[John, 2015/11/05 20:25:13]

Done.

+                                'specified target.')
     argparser.add_argument('--verbose', '-v', dest='verbose',
                            action='store_true',
                            help='Display some extra debugging output')
     argparser.add_argument('--sz', dest='sz_args', action='append', default=[],
                            help='Extra arguments for Subzero')
     argparser.add_argument('--llc', dest='llc_args', action='append',
                            default=[], help='Extra arguments for llc')
 
 def main():
     """Create a hybrid translation from Subzero and llc.
@@ skipping 77 matching lines @@
     llcbin = '{base}/pnacl-llc'.format(base=path_addition)
     gold = 'le32-nacl-ld.gold'
     objcopy = 'le32-nacl-objcopy'
     opt_level = args.optlevel
     opt_level_map = { 'm1':'0', '-1':'0', '0':'0', '1':'1', '2':'2' }
     hybrid = args.include or args.exclude
 
     if hybrid and (args.force or
                    NewerThanOrNotThere(pexe, obj_llc) or
                    NewerThanOrNotThere(llcbin, obj_llc)):
+        arch = {
+          'arm32': 'armv7' if args.sandbox else 'arm-nonsfi',
+          'x8632': 'x86-32' if args.sandbox else 'x86-32-linux',
+        }[args.target]
+
         # Only run pnacl-translate in hybrid mode.
         shellcmd(['pnacl-translate',
                   '-split-module=1',
                   '-ffunction-sections',
                   '-fdata-sections',
                   '-c',
-                  '-arch', 'x86-32' if args.sandbox else 'x86-32-linux',
+                  '-arch',  arch,
                   '-O' + opt_level_map[opt_level],
                   '--pnacl-driver-append-LLC_FLAGS_EXTRA=-externalize',
                   '-o', obj_llc] +
                  (['--pnacl-driver-verbose'] if args.verbose else []) +
                  args.llc_args +
                  [pexe],
                  echo=args.verbose)
         if not args.sandbox:
             shellcmd((
                 '{objcopy} --redefine-sym _start=_user_start {obj}'
                 ).format(objcopy=objcopy, obj=obj_llc), echo=args.verbose)
         # Generate llc syms file for consistency, even though it's not used.
         shellcmd((
             'nm {obj} | sed -n "s/.* [a-zA-Z] //p" > {sym}'
             ).format(obj=obj_llc, sym=sym_llc), echo=args.verbose)
 
     if (args.force or
         NewerThanOrNotThere(pexe, obj_sz) or
         NewerThanOrNotThere(pnacl_sz, obj_sz)):
         # Run pnacl-sz regardless of hybrid mode.
         shellcmd([pnacl_sz,
                   '-O' + opt_level,
                   '-bitcode-format=pnacl',
                   '-filetype=' + args.filetype,
-                  '-o', obj_sz if args.filetype == 'obj' else asm_sz] +
+                  '-o', obj_sz if args.filetype == 'obj' else asm_sz,
+                  '-target=' + args.target] +
                  (['-externalize',
                    '-ffunction-sections',
                    '-fdata-sections'] if hybrid else []) +
                  (['-sandbox'] if args.sandbox else []) +
                  (['-enable-block-profile'] if
                       args.enable_block_profile and not args.sandbox else []) +
                  args.sz_args +
                  [pexe],
                  echo=args.verbose)
         if args.filetype != 'obj':
+            triple = {
+              'arm32': 'arm-nacl' if args.sandbox else 'arm',
+              'x8632': 'i686-nacl' if args.sandbox else 'i686',
+            }[args.target]
+
             shellcmd((
                 'llvm-mc -triple={triple} -filetype=obj -o {obj} {asm}'
-                ).format(asm=asm_sz, obj=obj_sz,
-                         triple='i686-nacl' if args.sandbox else 'i686'),
+                ).format(asm=asm_sz, obj=obj_sz, triple=triple),
                      echo=args.verbose)
         if not args.sandbox:
             shellcmd((
                 '{objcopy} --redefine-sym _start=_user_start {obj}'
                 ).format(objcopy=objcopy, obj=obj_sz), echo=args.verbose)
         if hybrid:
             shellcmd((
                 'nm {obj} | sed -n "s/.* [a-zA-Z] //p" > {sym}'
                 ).format(obj=obj_sz, sym=sym_sz), echo=args.verbose)
 
@@ skipping 26 matching lines @@
                 ).format(objcopy=objcopy,
                          whitelist=whitelist_sz, obj=obj_llc,
                          weak=obj_llc_weak),
                      echo=args.verbose)
         else:
             shellcmd((
                 '{objcopy} {obj} {weak}'
                 ).format(objcopy=objcopy, obj=obj_llc, weak=obj_llc_weak),
                 echo=args.verbose)
         obj_partial = pexe_base + '.o'
+        ld = {
+          'arm32': 'arm-linux-gnueabihf-ld',
+          'x8632': 'ld',
+        }[args.target]
+        emulation = {
+          'arm32': 'armelf_linux_eabi',
+          'x8632': 'elf_i386',
+        }[args.target]
         shellcmd((
-            'ld -r -m elf_i386 -o {partial} {sz} {llc}'
-            ).format(partial=obj_partial, sz=obj_sz_weak, llc=obj_llc_weak),
+            '{ld} -r -m {emulation} -o {partial} {sz} {llc}'
+            ).format(ld=ld, emulation=emulation, partial=obj_partial,
+                     sz=obj_sz_weak, llc=obj_llc_weak),
                  echo=args.verbose)
         shellcmd((
             '{objcopy} -w --localize-symbol="*" {partial}'
             ).format(objcopy=objcopy, partial=obj_partial),
             echo=args.verbose)
         shellcmd((
             '{objcopy} --globalize-symbol={start} ' +
             '--globalize-symbol=__Sz_block_profile_info {partial}'
             ).format(objcopy=objcopy, partial=obj_partial,
                      start='_start' if args.sandbox else '_user_start'),
                  echo=args.verbose)
 
     # Run the linker regardless of hybrid mode.
-    linker = (
-        '{root}/../third_party/llvm-build/Release+Asserts/bin/clang'
-        ).format(root=nacl_root)
     if args.sandbox:
+        assert args.target in ['x8632'], \
+            '-sandbox is not available for %s' % args.target
         linklib = ('{root}/toolchain/linux_x86/pnacl_newlib_raw/translator/' +
                    'x86-32/lib').format(root=nacl_root)
         shellcmd((
             '{gold} -nostdlib --no-fix-cortex-a8 --eh-frame-hdr -z text ' +
             '--build-id --entry=__pnacl_start -static ' +
             '{linklib}/crtbegin.o {partial} ' +
             '{root}/toolchain_build/src/subzero/build/runtime/' +
-            'szrt_sb_x8632.o ' +
+            'szrt_sb_{target}.o ' +
             '{linklib}/libpnacl_irt_shim_dummy.a --start-group ' +
             '{linklib}/libgcc.a {linklib}/libcrt_platform.a ' +
             '--end-group {linklib}/crtend.o --undefined=_start ' +
             '--defsym=__Sz_AbsoluteZero=0 ' +
             '-o {exe}'
             ).format(gold=gold, linklib=linklib, partial=obj_partial, exe=exe,
-                     root=nacl_root),
+                     root=nacl_root, target=args.target),
                  echo=args.verbose)
     else:
+        linker = {
+          'arm32': '/usr/bin/arm-linux-gnueabihf-g++',
+          'x8632': ('{root}/../third_party/llvm-build/Release+Asserts/bin/clang'
+                   ).format(root=nacl_root)
+        }[args.target]
+
+        extra_linker_args = ' '.join({
+          'arm32': ['-mcpu=cortex-a9'],
+          'x8632': ['-m32']
+        }[args.target])
+
+        lib_dir = {
+          'arm32': 'arm-linux',
+          'x8632': 'x86-32-linux',
+        }[args.target]
+
         shellcmd((
-            '{ld} -m32 {partial} -o {exe} ' +
+            '{ld} {ld_extra_args} {partial} -o {exe} ' +
             # Keep the rest of this command line (except szrt_native_x8632.o) in
             # sync with RunHostLD() in pnacl-translate.py.
             '{root}/toolchain/linux_x86/pnacl_newlib_raw/translator/' +
-            'x86-32-linux/lib/' +
+            '{lib_dir}/lib/' +
             '{{unsandboxed_irt,irt_random,irt_query_list}}.o ' +
             '{root}/toolchain_build/src/subzero/build/runtime/' +
-            'szrt_native_x8632.o -lpthread -lrt ' +
+            'szrt_native_{target}.o -lpthread -lrt ' +
             '-Wl,--defsym=__Sz_AbsoluteZero=0'
-            ).format(ld=linker, partial=obj_partial, exe=exe, root=nacl_root),
+            ).format(ld=linker, ld_extra_args=extra_linker_args,
+                     partial=obj_partial, exe=exe, root=nacl_root,
+                     target=args.target, lib_dir=lib_dir),
                  echo=args.verbose)
 
     # Put the extra verbose printing at the end.
     if args.verbose:
         print 'PATH: {path}'.format(path=path_addition)
         if hybrid:
             print 'include={regex}'.format(regex=re_include_str)
             print 'exclude={regex}'.format(regex=re_exclude_str)
             print 'default_match={dm}'.format(dm=default_match)
             print 'Number of Subzero syms = {num}'.format(num=len(sz_syms))
 
 if __name__ == '__main__':
     main()