summaryrefslogtreecommitdiff
path: root/compiler/optimizing/instruction_builder.cc
diff options
context:
space:
mode:
author Santiago Aboy Solanes <solanes@google.com> 2024-03-22 10:45:15 +0000
committer Santiago Aboy Solanes <solanes@google.com> 2024-03-25 13:43:14 +0000
commit56dcaeec46dd53e271d6f201d2028996bf19f9dd (patch)
treeea607494f1912d60e838be94919e25db2ec3ac89 /compiler/optimizing/instruction_builder.cc
parent9c4a2b5bb3695349e31a00492731e578e8853a21 (diff)
Remove extra uses of ClearAllBits
ArenaBitVector creation guarantees it starts empty. Add a debug check to make sure this assumption doesn't change. Note that ArenaAllocator guarantees zero-initialized memory but ScopedArenaAllocators do not. This is fine either way since the BitVector constructor calls ClearAllBits. Bug: 329037671 Test: art/test/testrunner/testrunner.py --host --64 --optimizing -b Change-Id: Icbf5e5dd1869e80b5d5828ecca9f13de30c0242b
Diffstat (limited to 'compiler/optimizing/instruction_builder.cc')
-rw-r--r--compiler/optimizing/instruction_builder.cc1
1 files changed, 0 insertions, 1 deletions
diff --git a/compiler/optimizing/instruction_builder.cc b/compiler/optimizing/instruction_builder.cc
index 410d6fd0d0..1d665b2559 100644
--- a/compiler/optimizing/instruction_builder.cc
+++ b/compiler/optimizing/instruction_builder.cc
@@ -538,7 +538,6 @@ ArenaBitVector* HInstructionBuilder::FindNativeDebugInfoLocations() {
code_item_accessor_.InsnsSizeInCodeUnits(),
/* expandable= */ false,
kArenaAllocGraphBuilder);
- locations->ClearAllBits();
// The visitor gets called when the line number changes.
// In other words, it marks the start of new java statement.
code_item_accessor_.DecodeDebugPositionInfo([&](const DexFile::PositionInfo& entry) {