diff options
| author |  Benjamin Peterson <bp@benjamin.pe> | 2017-11-27 03:14:43 -0800 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2017-11-27 03:16:07 -0800 |
| commit | 8610d97d2b68cfbcf6b7de4441665a64f378eb9c (patch) | |
| tree | 15a4967abd18a1963fa147fcbd462ca74155085d /src/main/java/com/google/devtools/build/lib/analysis | |
| parent | 9fea5b89a5cf8744dbe32aff279a9e2c2613c9d8 (diff) | |
Shape sharing for Skylark providers.
Add CompactSkylarkInfo, which stores its values as an array instead of
a map. The space savings will probably not be dramatic because
providers usually have a limited amount of keys. But, there are a lot
of them!
Change-Id: Idd452a5e3982f773b1c5202c73f3d7031ec022c6
PiperOrigin-RevId: 176995376
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java b/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java index ac7ade351c..429f419b2c 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/ActionsProvider.java @@ -46,6 +46,6 @@ public final class ActionsProvider { } } ImmutableMap<String, Object> fields = ImmutableMap.<String, Object>of("by_file", map); - return new SkylarkInfo(SKYLARK_CONSTRUCTOR, fields, Location.BUILTIN); + return SkylarkInfo.fromMap(SKYLARK_CONSTRUCTOR, fields, Location.BUILTIN); } } |