diff options
| author | 2016-03-02 16:48:20 +0000 | |
|---|---|---|
| committer | 2016-04-04 11:21:30 +0100 | |
| commit | e3ff7b293be2a6791fe9d135d660c0cffe4bd73f (patch) | |
| tree | d578d27cb78e6d2caef683cd8ac94c9a9752b192 /compiler/optimizing/optimizing_compiler.cc | |
| parent | 86ea7eeabe30c98bbe1651a51d03cb89776724e7 (diff) | |
Refactor HGraphBuilder and SsaBuilder to remove HLocals
This patch merges the instruction-building phases from HGraphBuilder
and SsaBuilder into a single HInstructionBuilder class. As a result,
it is not necessary to generate HLocal, HLoadLocal and HStoreLocal
instructions any more, as the builder produces SSA form directly.
Saves 5-15% of arena-allocated memory (see bug for more data):
GMS 20.46MB => 19.26MB (-5.86%)
Maps 24.12MB => 21.47MB (-10.98%)
YouTube 28.60MB => 26.01MB (-9.05%)
Bug: 27894376
Change-Id: Iefe28d40600c169c5d306fd2c77034ae19476d90
Diffstat (limited to 'compiler/optimizing/optimizing_compiler.cc')
| -rw-r--r-- | compiler/optimizing/optimizing_compiler.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/compiler/optimizing/optimizing_compiler.cc b/compiler/optimizing/optimizing_compiler.cc index 20a666128f..3d6bf62d0b 100644 --- a/compiler/optimizing/optimizing_compiler.cc +++ b/compiler/optimizing/optimizing_compiler.cc @@ -731,8 +731,9 @@ CodeGenerator* OptimizingCompiler::TryCompile(ArenaAllocator* arena, compiler_driver, compilation_stats_.get(), interpreter_metadata, - dex_cache); - GraphAnalysisResult result = builder.BuildGraph(&handles); + dex_cache, + &handles); + GraphAnalysisResult result = builder.BuildGraph(); if (result != kAnalysisSuccess) { switch (result) { case kAnalysisSkipped: |