diff options
author | Richard Uhler <ruhler@google.com> | 2016-03-18 15:05:30 -0700 |
---|---|---|
committer | Richard Uhler <ruhler@google.com> | 2016-03-22 15:38:36 -0700 |
commit | a62d2f04a6ecf804f8a78e722a6ca8ccb2dfa931 (patch) | |
tree | d8624bb85016a3301b473b6503324db4d4cf24a3 /compiler/driver/compiler_options.h | |
parent | 48e722432bb6e19df7bba02427e4a707e671af06 (diff) |
Use compiler filter to determine oat file status.
Record the compiler filter in the oat header. Use that to determine
when the oat file is up-to-date with respect to a target compiler
filter level.
New xxx-profile filter levels are added to specify if a profile should
be used instead of testing for the presence of a profile file.
This change should allow for different compiler-filters to be set for
different package manager use cases.
Bug: 27689078
Change-Id: Id6706d0ed91b45f307142692ea4316aa9713b023
Diffstat (limited to 'compiler/driver/compiler_options.h')
-rw-r--r-- | compiler/driver/compiler_options.h | 42 |
1 files changed, 11 insertions, 31 deletions
diff --git a/compiler/driver/compiler_options.h b/compiler/driver/compiler_options.h index 11a4e06313..c67ab6ef14 100644 --- a/compiler/driver/compiler_options.h +++ b/compiler/driver/compiler_options.h @@ -22,6 +22,7 @@ #include <vector> #include "base/macros.h" +#include "compiler_filter.h" #include "globals.h" #include "utils.h" @@ -29,20 +30,8 @@ namespace art { class CompilerOptions FINAL { public: - enum CompilerFilter { - kVerifyNone, // Skip verification and compile nothing except JNI stubs. - kInterpretOnly, // Verify, and compile only JNI stubs. - kVerifyAtRuntime, // Only compile JNI stubs and verify at runtime. - kSpace, // Maximize space savings. - kBalanced, // Try to get the best performance return on compilation investment. - kSpeed, // Maximize runtime performance. - kEverything, // Force compilation of everything capable of being compiled. - kTime, // Compile methods, but minimize compilation time. - kVerifyProfile, // Verify only the classes in the profile. - }; - // Guide heuristics to determine whether to compile method if profile data not available. - static const CompilerFilter kDefaultCompilerFilter = kSpeed; + static const CompilerFilter::Filter kDefaultCompilerFilter = CompilerFilter::kSpeed; static const size_t kDefaultHugeMethodThreshold = 10000; static const size_t kDefaultLargeMethodThreshold = 600; static const size_t kDefaultSmallMethodThreshold = 60; @@ -64,7 +53,7 @@ class CompilerOptions FINAL { CompilerOptions(); ~CompilerOptions(); - CompilerOptions(CompilerFilter compiler_filter, + CompilerOptions(CompilerFilter::Filter compiler_filter, size_t huge_method_threshold, size_t large_method_threshold, size_t small_method_threshold, @@ -88,40 +77,32 @@ class CompilerOptions FINAL { bool dump_cfg_append, bool force_determinism); - CompilerFilter GetCompilerFilter() const { + CompilerFilter::Filter GetCompilerFilter() const { return compiler_filter_; } - void SetCompilerFilter(CompilerFilter compiler_filter) { + void SetCompilerFilter(CompilerFilter::Filter compiler_filter) { compiler_filter_ = compiler_filter; } bool VerifyAtRuntime() const { - return compiler_filter_ == CompilerOptions::kVerifyAtRuntime; + return compiler_filter_ == CompilerFilter::kVerifyAtRuntime; } bool IsCompilationEnabled() const { - return compiler_filter_ != CompilerOptions::kVerifyNone && - compiler_filter_ != CompilerOptions::kInterpretOnly && - compiler_filter_ != CompilerOptions::kVerifyAtRuntime && - compiler_filter_ != CompilerOptions::kVerifyProfile; + return CompilerFilter::IsCompilationEnabled(compiler_filter_); } bool IsVerificationEnabled() const { - return compiler_filter_ != CompilerOptions::kVerifyNone && - compiler_filter_ != CompilerOptions::kVerifyAtRuntime; + return CompilerFilter::IsVerificationEnabled(compiler_filter_); } bool NeverVerify() const { - return compiler_filter_ == CompilerOptions::kVerifyNone; - } - - bool IsExtractOnly() const { - return compiler_filter_ == CompilerOptions::kVerifyAtRuntime; + return compiler_filter_ == CompilerFilter::kVerifyNone; } bool VerifyOnlyProfile() const { - return compiler_filter_ == CompilerOptions::kVerifyProfile; + return compiler_filter_ == CompilerFilter::kVerifyProfile; } size_t GetHugeMethodThreshold() const { @@ -271,7 +252,7 @@ class CompilerOptions FINAL { void ParseLargeMethodMax(const StringPiece& option, UsageFn Usage); void ParseHugeMethodMax(const StringPiece& option, UsageFn Usage); - CompilerFilter compiler_filter_; + CompilerFilter::Filter compiler_filter_; size_t huge_method_threshold_; size_t large_method_threshold_; size_t small_method_threshold_; @@ -317,7 +298,6 @@ class CompilerOptions FINAL { DISALLOW_COPY_AND_ASSIGN(CompilerOptions); }; -std::ostream& operator<<(std::ostream& os, const CompilerOptions::CompilerFilter& rhs); } // namespace art |