commit 698ebbca3c54d17c696e87b9e5838df6a5ac9b08
author Mathieu Chartier <mathieuc@google.com> Fri Jan 05 11:00:42 2018 -0800
committer Mathieu Chartier <mathieuc@google.com> Mon Jan 08 09:11:49 2018 -0800
tree 7887b2cd4662410114f205bbe1896ff094d51407
parent 4b9a11f1d43df69c50e93ba1793fce1c2216682b

Clean up CodeItemAccessors and Compact/StandardDexFile

Change constructor to use a reference to a dex file.

Remove duplicated logic for GetCodeItemSize.

Bug: 63756964
Test: test-art-host
Change-Id: I69af8b93abdf6bdfa4454e16db8f4e75883bca46

compiler/optimizing/optimization.cc

diff --git a/compiler/optimizing/optimization.cc b/compiler/optimizing/optimization.cc
index 92b427c..57db7a6 100644
--- a/compiler/optimizing/optimization.cc
+++ b/compiler/optimizing/optimization.cc
@@ -242,7 +242,7 @@
         opt = new (allocator) HDeadCodeElimination(graph, stats, name);
         break;
       case OptimizationPass::kInliner: {
-        CodeItemDataAccessor accessor(dex_compilation_unit.GetDexFile(),
+        CodeItemDataAccessor accessor(*dex_compilation_unit.GetDexFile(),
                                       dex_compilation_unit.GetCodeItem());
         opt = new (allocator) HInliner(graph,                   // outer_graph
                                        graph,                   // outermost_graph