Subzero, MIPS32: Cross-testing enabled for MIPS32

pydir/build-runtime.py

Index: pydir/build-runtime.py
diff --git a/pydir/build-runtime.py b/pydir/build-runtime.py
index caed29722f285d69ea552ac3d1f414aad864e5c8..2f0387c1ddd248abef2b36e5d373e5153955982c 100755
--- a/pydir/build-runtime.py
+++ b/pydir/build-runtime.py
@@ -9,7 +9,7 @@ import targets
 from utils import FindBaseNaCl, GetObjcopyCmd, shellcmd
 
 
-def Translate(ll_files, extra_args, obj, verbose):
+def Translate(ll_files, extra_args, obj, verbose, target):
   """Translate a set of input bitcode files into a single object file.
 
   Use pnacl-llc to translate textual bitcode input ll_files into object file
@@ -17,7 +17,6 @@ def Translate(ll_files, extra_args, obj, verbose):
   """
   shellcmd(['cat'] + ll_files + ['|',
             'pnacl-llc',
-            '-externalize',
             '-function-sections',
             '-O2',
             '-filetype=obj',
@@ -25,7 +24,7 @@ def Translate(ll_files, extra_args, obj, verbose):
             '-arm-enable-dwarf-eh=1',
             '-o', obj
     ] + extra_args, echo=verbose)
-  shellcmd([GetObjcopyCmd(),
+  shellcmd([GetObjcopyCmd(target),
             '--strip-symbol=nacl_tp_tdb_offset',
             '--strip-symbol=nacl_tp_tls_offset',
             obj
@@ -56,7 +55,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
     Translate(ll_files,
               ['-mtriple=' + target_info.triple] + target_info.llc_flags,
               TmpFile('{dir}/szrt_native_{target}.tmp.o'),
-              verbose)
+              verbose,target=target_info.target)
     # Compile srcdir/szrt_profiler.c to
     # tempdir/szrt_profiler_native_{target}.o.
     shellcmd(['clang',
@@ -65,7 +64,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
               '-c',
               '{srcdir}/szrt_profiler.c'.format(srcdir=srcdir),
               '-o', TmpFile('{dir}/szrt_native_profiler_{target}.o')
-      ], echo=verbose)
+            ], echo=verbose)

[Jim Stichnoth, 2016/08/23 14:56:48]

revert this whitespace change

[obucinac, 2016/09/05 16:55:59]

Done.

     # Assemble srcdir/szrt_asm_{target}.s to tempdir/szrt_asm_{target}.o.
     shellcmd(['llvm-mc',
               '-triple=' + target_info.triple, '--defsym NATIVE=1',
@@ -81,7 +80,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
                 ['-m {ld_emu}'.format(ld_emu=target_info.ld_emu)],
                 OutFile('{rtdir}/szrt_native_{target}.o'),
                 verbose)
-    shellcmd([GetObjcopyCmd(),
+    shellcmd([GetObjcopyCmd(target=target_info.target),

[Jim Stichnoth, 2016/08/23 14:56:48]

Can you remove "target=" ?

here and below

[obucinac, 2016/09/05 16:55:59]

Done.

               '--strip-symbol=NATIVE',
               OutFile('{rtdir}/szrt_native_{target}.o')])
     # Compile srcdir/szrt_asan.c to szrt_asan_{target}.o
@@ -103,7 +102,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
               ['-mtriple=' + targets.ConvertTripleToNaCl(target_info.triple)] +
               target_info.llc_flags,
               TmpFile('{dir}/szrt_sb_{target}.tmp.o'),
-              verbose)
+              verbose,target=target_info.target)
     # Assemble srcdir/szrt_asm_{target}.s to tempdir/szrt_asm_{target}.o.
     shellcmd(['llvm-mc',
               '-triple=' + targets.ConvertTripleToNaCl(target_info.triple),
@@ -118,7 +117,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
                 ['-m {ld_emu}'.format(ld_emu=target_info.sb_emu)],
                 OutFile('{rtdir}/szrt_sb_{target}.o'),
                 verbose)
-    shellcmd([GetObjcopyCmd(),
+    shellcmd([GetObjcopyCmd(target=target_info.target),
               '--strip-symbol=NACL',
               OutFile('{rtdir}/szrt_sb_{target}.o')])
 
@@ -131,7 +130,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
               ['-mtriple=' + target_info.triple] + target_info.llc_flags +
               ['-relocation-model=pic', '-force-tls-non-pic', '-malign-double'],
               TmpFile('{dir}/szrt_nonsfi_{target}.tmp.o'),
-              verbose)
+              verbose, target=target_info.target)
     # Assemble srcdir/szrt_asm_{target}.s to tempdir/szrt_asm_{target}.o.
     shellcmd(['llvm-mc',
               '-triple=' + target_info.triple, '--defsym NONSFI=1',
@@ -146,7 +145,7 @@ def MakeRuntimesForTarget(target_info, ll_files,
                 ['-m {ld_emu}'.format(ld_emu=target_info.ld_emu)],
                 OutFile('{rtdir}/szrt_nonsfi_{target}.o'),
                 verbose)
-    shellcmd([GetObjcopyCmd(),
+    shellcmd([GetObjcopyCmd(target=target_info.target),
               '--strip-symbol=NONSFI',
               OutFile('{rtdir}/szrt_nonsfi_{target}.o')])
 
@@ -211,6 +210,8 @@ def main():
                               srcdir, tempdir, rtdir, args.verbose)
         MakeRuntimesForTarget(targets.ARM32Target, ll_files,
                               srcdir, tempdir, rtdir, args.verbose)
+        MakeRuntimesForTarget(targets.MIPS32Target, ll_files,

[Jim Stichnoth, 2016/09/02 15:49:31]

We need a way to not run this in a non-MIPS build environment.

It's probably best to control that via a command-line flag in this script.

Maybe add something like this above:

    argparser.add_argument('--exclude-target', dest='exclude_target',
                           default=[], action='append',
                           help='Target whose runtime should not be built')

And then pass args.exclude_target as an additional argument to
MakeRuntimesForTarget().  And at the top of MakeRuntimesForTarget(), do an early
exit:

  if target_info.target in excluded_targets:
    return

Then the Makefile.standalone needs some special MIPS controls for RT_SRC and
RT_OBJ.

I think this could be effectively tested for non-MIPS by temporarily defining
FindMIPSCrossInclude() to glob a nonexistent directory.

+                              srcdir, tempdir, rtdir, args.verbose)
 
     finally:
         try: