mirror of
https://github.com/dart-lang/sdk
synced 2024-09-05 00:13:50 +00:00
Adds a flag to run_binary_size_analysis.py to disable format checks
This change is needed for compatibility with the Flutter Engine's use of a new Android NDK. The new Android NDK includes a new clang toolchain which does a better job of classifying various contributors to a binary's size, but which fails the compatibility checks in the script. The intention of the compatibility checks is to exclude versions of tooling that are too old not versions that are too new. The compatibility check is also redundant with failure modes later in the script. To avoid breaking any usage of this script in the Dart tree, however, the check is retained by this change, and instead a flag is added that skips it. This is preferable to expanding on the questionable use of regular expressions that implement the check to capture the strings emitted by a continually updating toolchain. Change-Id: If379e4aa86339a31eada0b6ce3fa90ddc9351804 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/337602 Commit-Queue: Zach Anderson <zra@google.com> Reviewed-by: Ryan Macnak <rmacnak@google.com> Reviewed-by: Paul Berry <paulberry@google.com> Reviewed-by: Siva Annamalai <asiva@google.com>
This commit is contained in:
parent
2f30295254
commit
8724025782
|
@ -620,6 +620,16 @@ def main():
|
|||
help='the path to the source code of the output binary, '
|
||||
'default set to current directory. Used in the'
|
||||
' disambiguation process.')
|
||||
parser.add_option(
|
||||
'--check-support',
|
||||
dest='check_support',
|
||||
default=True,
|
||||
action='store_true',
|
||||
help='Check that the version of the available tools is sufficient to '
|
||||
'read the data from the library given by --library')
|
||||
parser.add_option('--no-check-support',
|
||||
action='store_false',
|
||||
dest='check_support')
|
||||
opts, _args = parser.parse_args()
|
||||
|
||||
if ((not opts.library) and
|
||||
|
@ -663,7 +673,7 @@ def main():
|
|||
print('addr2line: %s' % addr2line_binary)
|
||||
print('nm: %s' % nm_binary)
|
||||
|
||||
if opts.library:
|
||||
if opts.library and opts.check_support:
|
||||
CheckDebugFormatSupport(opts.library, addr2line_binary)
|
||||
|
||||
# Prepare output directory and report guts
|
||||
|
|
Loading…
Reference in a new issue