Skip to content

Commit

Permalink
Remove the flutter_aot GN argument. (flutter#8581)
Browse files Browse the repository at this point in the history
This argument has been defunct for a while. It was originally added as a
means of testing the AOT machinery in debug modes.
  • Loading branch information
chinmaygarde authored Apr 15, 2019
1 parent 571ce94 commit 324b840
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
7 changes: 0 additions & 7 deletions common/config.gni
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,6 @@ if (target_cpu == "arm" || target_cpu == "arm64") {
}

declare_args() {
# Enable ahead-of-time compilation on platforms where AOT is optional.
flutter_aot = false

# The runtime mode ("debug", "profile", "release", "dynamic_profile", or "dynamic_release")
flutter_runtime_mode = "debug"
}
Expand Down Expand Up @@ -41,7 +38,3 @@ if (flutter_runtime_mode == "debug") {
} else {
feature_defines_list += [ "FLUTTER_RUNTIME_MODE=0" ]
}

if (flutter_aot) {
feature_defines_list += [ "FLUTTER_AOT=1" ]
}
6 changes: 0 additions & 6 deletions tools/gn
Original file line number Diff line number Diff line change
Expand Up @@ -110,18 +110,13 @@ def to_gn_args(args):
# The GN arg is not available in the windows toolchain.
gn_args['enable_lto'] = enable_lto

aot = not runtime_mode in ['debug', 'dynamic_profile', 'dynamic_release']
if args.target_os == 'android':
gn_args['target_os'] = 'android'
elif args.target_os == 'ios':
gn_args['target_os'] = 'ios'
gn_args['use_ios_simulator'] = args.simulator
if not args.simulator:
aot = True
elif args.target_os is not None:
gn_args['target_os'] = args.target_os
else:
aot = False

gn_args['dart_lib_export_symbols'] = False

Expand Down Expand Up @@ -189,7 +184,6 @@ def to_gn_args(args):
gn_args["host_cpu"] = "x86"

gn_args['flutter_runtime_mode'] = runtime_mode
gn_args['flutter_aot'] = aot

if args.target_sysroot:
gn_args['target_sysroot'] = args.target_sysroot
Expand Down

0 comments on commit 324b840

Please sign in to comment.