| commit | cc31ff2983a950c5cb72a439389d2faea597d55d | [log] [tgz] |
|---|---|---|
| author | Googler <noreply@google.com> | Tue Dec 03 07:05:55 2019 -0800 |
| committer | Copybara-Service <copybara-worker@google.com> | Tue Dec 03 07:06:59 2019 -0800 |
| tree | bd998ac64d1902a130d46b89e7c33664003cb5c2 | |
| parent | 83d7e1d27a84f2db0d6197af50955e26a9db04fb [diff] |
bazel nestedset: don't allocate empty CompactHashSet in addAll commit 565c199f51cf8b9898342c47c07666c1a8159375 recently changed the depset constructor to use NestedSetBuilder.addAll, which should be more efficient than a loop around add, but it was not, resulting in a measurable memory regression: addAll causes the allocation of a CompactHashSet even when there are no elements. This change fixes the bug. Thanks to brandjon for spotting the problem. PiperOrigin-RevId: 283537882
{Fast, Correct} - Choose two
Build and test software of any size, quickly and reliably.
Speed up your builds and tests: Bazel rebuilds only what is necessary. With advanced local and distributed caching, optimized dependency analysis and parallel execution, you get fast and incremental builds.
One tool, multiple languages: Build and test Java, C++, Android, iOS, Go, and a wide variety of other language platforms. Bazel runs on Windows, macOS, and Linux.
Scalable: Bazel helps you scale your organization, codebase, and continuous integration solution. It handles codebases of any size, in multiple repositories or a huge monorepo.
Extensible to your needs: Easily add support for new languages and platforms with Bazel's familiar extension language. Share and re-use language rules written by the growing Bazel community.
Follow our tutorials:
See CONTRIBUTING.md
