diff options
| author | Ulf Magnusson <ulfalizer@gmail.com> | 2018-07-11 10:25:41 +0200 |
|---|---|---|
| committer | Ulf Magnusson <ulfalizer@gmail.com> | 2018-07-11 14:55:25 +0200 |
| commit | 71872a88641f713dfa986906e860c320a74fd84a (patch) | |
| tree | 0249602317d3ce2c6f73b2c015c01e84d534ef26 /allnoconfig.py | |
| parent | f0a8a4d86e4c52b401f543737d76ae8392247300 (diff) | |
Massively speed up U-Boot parsing
U-Boot has a ton of definition locations for some symbols, causing a lot
of redundant work when iterating over Kconfig.defined_syms in
_build_dep(). Iterate over set(Kconfig.defined_syms) instead, wherever
possible.
This speeds up the U-Boot parsing time from 4 seconds to 0.6 seconds on
my machine.
Also update the bundled tools to iterate over set(Kconfig.defined_syms).
The performance loss is negligible even for projects that don't use
multiple definition locations.
Update the documentation to clarify that symbols/choices defined in
multiple locations appear multiple times in Kconfig.defined_syms/choices
as well.
Diffstat (limited to 'allnoconfig.py')
| -rwxr-xr-x | allnoconfig.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/allnoconfig.py b/allnoconfig.py index d6ab701..173bf3f 100755 --- a/allnoconfig.py +++ b/allnoconfig.py @@ -31,7 +31,9 @@ def main(): # symbol types, which is what we want. kconf.disable_warnings() - for sym in kconf.defined_syms: + # The set() speeds things up for projects that use multiple definition + # locations a lot + for sym in set(kconf.defined_syms): sym.set_value(2 if sym.is_allnoconfig_y else 0) kconf.write_config(kconfiglib.standard_config_filename()) |
