From 56dcaeec46dd53e271d6f201d2028996bf19f9dd Mon Sep 17 00:00:00 2001 From: Santiago Aboy Solanes Date: Fri, 22 Mar 2024 10:45:15 +0000 Subject: 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 --- compiler/optimizing/graph_checker.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'compiler/optimizing/graph_checker.h') 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 -- cgit v1.2.3-59-g8ed1b