From 31242a99a1b58ce5267c1a74ad96a9d0b6dc1a94 Mon Sep 17 00:00:00 2001 From: Santiago Aboy Solanes Date: Mon, 5 Dec 2022 14:38:00 +0000 Subject: Update domination chain and RPO manually in MaybeAddExtraGotoBlocks There's no need to recompute the whole graph as we know what changed. As a drive-by, we now don't return false for graphs with irreducible loops so we can remove that restriction from the builder. However, if a graph with irreducible loops hits this path it means that: A) it's being inlined B) Has irreducible loops We don't inline graphs with irreducible loops, and after building for inline we don't remove them either because constant folding and instruction simplifier don't remove them, and DCE doesn't run for graphs with irreducible loops. So, in terms of dex2oat's outputs nothing should change. Test: art/test/testrunner/testrunner.py --host --64 --optimizing -b Change-Id: I8cbf1b5f0518bb5dd14ffd751100ea81f5478863 --- compiler/optimizing/builder.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'compiler/optimizing/builder.h') diff --git a/compiler/optimizing/builder.h b/compiler/optimizing/builder.h index 85fa8d4123..145dbfb6b9 100644 --- a/compiler/optimizing/builder.h +++ b/compiler/optimizing/builder.h @@ -57,8 +57,7 @@ class HGraphBuilder : public ValueObject { // When inlining, we sometimes want to add an extra Goto block before the Exit block. This is done // in the building phase as we do not allow the inlining phase to add new instructions. - // Returns false if the graph we are adding the extra block has irreducible loops. - bool MaybeAddExtraGotoBlocks(); + void MaybeAddExtraGotoBlocks(); HGraph* const graph_; const DexFile* const dex_file_; -- cgit v1.2.3-59-g8ed1b