commit dce164adfbf679d7ec9c9dc778fa3fb596011740
author Ian Rogers <irogers@google.com> Thu Jan 02 17:00:38 2014 -0800
committer Ian Rogers <irogers@google.com> Thu Jan 02 17:01:44 2014 -0800
tree f7bafd73fd7eee1ff7eba3f1923e3bb534be95b4
parent a72a4b4d7d1d32913f44f07db7c5744905f81f54

Avoid inliner spam wrt String.length.

Also, remove a use of std::map for our preferred SafeMap and be more explicit
with types.

Change-Id: I7b9a4bb1f73c22490fe416503e050671e7c99fe0

compiler/dex/quick/dex_file_method_inliner.h

diff --git a/compiler/dex/quick/dex_file_method_inliner.h b/compiler/dex/quick/dex_file_method_inliner.h
index 1d99731..6e81303 100644
--- a/compiler/dex/quick/dex_file_method_inliner.h
+++ b/compiler/dex/quick/dex_file_method_inliner.h
@@ -18,9 +18,9 @@
 #define ART_COMPILER_DEX_QUICK_DEX_FILE_METHOD_INLINER_H_
 
 #include <stdint.h>
-#include <map>
 #include "base/mutex.h"
 #include "base/macros.h"
+#include "safe_map.h"
 #include "dex/compiler_enums.h"
 #include "dex_file.h"
 #include "locks.h"
@@ -56,14 +56,15 @@
   kInlineOpIPut,
 };
 
-enum InlineMethodFlags {
-  kInlineIntrinsic = 0x0001,
-  kInlineSpecial   = 0x0002,
+enum InlineMethodFlags : uint16_t {
+  kNoInlineMethodFlags = 0x0000,
+  kInlineIntrinsic     = 0x0001,
+  kInlineSpecial       = 0x0002,
 };
 
 struct InlineMethod {
-  uint16_t opcode;
-  uint16_t flags;
+  InlineMethodOpcode opcode;
+  InlineMethodFlags flags;
   uint32_t data;
 };
 
@@ -369,7 +370,7 @@
     friend class DexFileToMethodInlinerMap;
 
     bool AddInlineMethod(int32_t method_idx, InlineMethodOpcode opcode,
-                         uint16_t flags, uint32_t data) LOCKS_EXCLUDED(lock_);
+                         InlineMethodFlags flags, uint32_t data) LOCKS_EXCLUDED(lock_);
 
     bool AnalyseReturnMethod(int32_t method_idx, const DexFile::CodeItem* code_item,
                              OpSize size) LOCKS_EXCLUDED(lock_);
@@ -384,7 +385,7 @@
     /*
      * Maps method indexes (for the particular DexFile) to Intrinsic defintions.
      */
-    std::map<uint32_t, InlineMethod> inline_methods_ GUARDED_BY(lock_);
+    SafeMap<uint32_t, InlineMethod> inline_methods_ GUARDED_BY(lock_);
     const DexFile* dex_file_;
 
     DISALLOW_COPY_AND_ASSIGN(DexFileMethodInliner);