diff options
author | Igor Murashkin <iam@google.com> | 2017-08-09 10:38:31 -0700 |
---|---|---|
committer | Igor Murashkin <iam@google.com> | 2017-09-08 14:06:32 +0000 |
commit | dd018df8a00e841fe38fabe38520b7d297a885c1 (patch) | |
tree | 4974b4f718d9cbb5910df7f476f2581aff1a17e2 /compiler/optimizing/optimizing_compiler_stats.h | |
parent | 6ef45677305048c2bf0600f1c4b98a11b2cfaffb (diff) |
optimizing: add block-scoped constructor fence merging pass
Introduce a new "Constructor Fence Redundancy Elimination" pass.
The pass currently performs local optimization only, i.e. within instructions
in the same basic block.
All constructor fences preceding a publish (e.g. store, invoke) get
merged into one instruction.
==============
OptStat#ConstructorFenceGeneratedNew: 43825
OptStat#ConstructorFenceGeneratedFinal: 17631 <+++
OptStat#ConstructorFenceRemovedLSE: 164
OptStat#ConstructorFenceRemovedPFRA: 9391
OptStat#ConstructorFenceRemovedCFRE: 16133 <---
Removes ~91.5% of the 'final' constructor fences in RitzBenchmark:
(We do not distinguish the exact reason that a fence was created, so
it's possible some "new" fences were also removed.)
==============
Test: art/test/run-test --host --optimizing 476-checker-ctor-fence-redun-elim
Bug: 36656456
Change-Id: I8020217b448ad96ce9b7640aa312ae784690ad99
Diffstat (limited to 'compiler/optimizing/optimizing_compiler_stats.h')
-rw-r--r-- | compiler/optimizing/optimizing_compiler_stats.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/compiler/optimizing/optimizing_compiler_stats.h b/compiler/optimizing/optimizing_compiler_stats.h index d6da73cc1c..af7ab2f1a0 100644 --- a/compiler/optimizing/optimizing_compiler_stats.h +++ b/compiler/optimizing/optimizing_compiler_stats.h @@ -91,6 +91,7 @@ enum MethodCompilationStat { kConstructorFenceGeneratedFinal, kConstructorFenceRemovedLSE, kConstructorFenceRemovedPFRA, + kConstructorFenceRemovedCFRE, kLastStat }; @@ -211,6 +212,7 @@ class OptimizingCompilerStats { case kConstructorFenceGeneratedFinal: name = "ConstructorFenceGeneratedFinal"; break; case kConstructorFenceRemovedLSE: name = "ConstructorFenceRemovedLSE"; break; case kConstructorFenceRemovedPFRA: name = "ConstructorFenceRemovedPFRA"; break; + case kConstructorFenceRemovedCFRE: name = "ConstructorFenceRemovedCFRE"; break; case kLastStat: LOG(FATAL) << "invalid stat " |