Raise dx's max java heap 1gb->2gb (-Xmx)

build/android/gyp/dex.py

 #!/usr/bin/env python
 #
 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import json
 import logging
 import optparse
 import os
@@ skipping 193 matching lines @@
     exclude_paths = options.excluded_paths
     paths = [p for p in paths if not
              os.path.relpath(p, options.output_directory) in exclude_paths]
 
   input_paths = list(paths)
 
   dx_binary = os.path.join(options.android_sdk_tools, 'dx')
   # See http://crbug.com/272064 for context on --force-jumbo.
   # See https://github.com/android/platform_dalvik/commit/dd140a22d for
   # --num-threads.
-  dex_cmd = [dx_binary, '--num-threads=8', '--dex', '--force-jumbo',
+  # See http://crbug.com/658782 for why -JXmx2G was added.
+  dex_cmd = [dx_binary, '-JXmx2G', '--num-threads=8', '--dex', '--force-jumbo',

[jbudorick, 2016/10/24 20:46:06]

JX?

[agrieve, 2016/10/24 20:47:34]

Yeah, it's because "dx" is a bash script that wraps the call to java. There's a
comment in the script showing how to set jvm args (via -J)

[jbudorick, 2016/10/24 20:48:35]

sgtm

              '--output', options.dex_path]
   if options.no_locals != '0':
     dex_cmd.append('--no-locals')
 
   if options.multi_dex:
     input_paths.append(options.main_dex_list_path)
     dex_cmd += [
       '--multi-dex',
       '--minimal-main-dex',
     ]
@@ skipping 12 matching lines @@
       options,
       input_paths=input_paths,
       input_strings=dex_cmd,
       output_paths=output_paths,
       force=force,
       pass_changes=True)
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv[1:]))