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

1 files changed, 1 insertions, 3 deletions

diff --git a/compiler/optimizing/graph_checker.h b/compiler/optimizing/graph_checker.h
index 541a9cc3d2..a1b7b28a24 100644
--- a/compiler/optimizing/graph_checker.h
+++ b/compiler/optimizing/graph_checker.h
@@ -43,9 +43,7 @@ class GraphChecker final : public HGraphDelegateVisitor {
         uses_per_instruction_(allocator_.Adapter(kArenaAllocGraphChecker)),
         instructions_per_block_(allocator_.Adapter(kArenaAllocGraphChecker)),
         phis_per_block_(allocator_.Adapter(kArenaAllocGraphChecker)),
-        codegen_(codegen) {
-    seen_ids_.ClearAllBits();
-  }
+        codegen_(codegen) {}
 
   // Check the whole graph. The pass_change parameter indicates whether changes
   // may have occurred during the just executed pass. The default value is