commit 0571d357843c53e042f370f5f2c2e9aa3fe803a9
author Ian Rogers <irogers@google.com> Thu Nov 03 19:51:38 2011 -0700
committer Ian Rogers <irogers@google.com> Sun Nov 13 11:57:18 2011 -0800
tree 197e5b1c0b1bf0ee8cc0e1960470d0a8d203b4af
parent 33dc7717cd16592bcc825350bea6305be9eb2ea1

Refactor the use of Method by the compiler.

Remove the dependence on the Method object in dex2oat, allowing lazier
resolution.
Introduce new find and iterators in DexFile to simplify common
operations and avoid misuse of class data items.

Change-Id: I39fb8252190f543d89d8b233076355cec310fe08

src/common_test.h

diff --git a/src/common_test.h b/src/common_test.h
index 3966742..56b54a3 100644
--- a/src/common_test.h
+++ b/src/common_test.h
@@ -100,7 +100,9 @@
   void MakeExecutable(Method* method) {
     CHECK(method != NULL);
 
-    const CompiledInvokeStub* compiled_invoke_stub = compiler_->GetCompiledInvokeStub(method);
+    const CompiledInvokeStub* compiled_invoke_stub =
+        compiler_->FindInvokeStub(method->IsStatic(),
+                                  method->GetShorty()->ToModifiedUtf8().c_str());
     CHECK(compiled_invoke_stub != NULL) << PrettyMethod(method);
     const std::vector<uint8_t>& invoke_stub = compiled_invoke_stub->GetCode();
     MakeExecutable(invoke_stub);
@@ -110,7 +112,11 @@
               << " invoke_stub=" << reinterpret_cast<void*>(method_invoke_stub);
 
     if (!method->IsAbstract()) {
-      const CompiledMethod* compiled_method = compiler_->GetCompiledMethod(method);
+      const DexCache* dex_cache = method->GetDeclaringClass()->GetDexCache();
+      const DexFile& dex_file = Runtime::Current()->GetClassLinker()->FindDexFile(dex_cache);
+      const CompiledMethod* compiled_method =
+          compiler_->GetCompiledMethod(Compiler::MethodReference(&dex_file,
+                                                                 method->GetDexMethodIndex()));
       CHECK(compiled_method != NULL) << PrettyMethod(method);
       const std::vector<uint8_t>& code = compiled_method->GetCode();
       MakeExecutable(code);
@@ -119,7 +125,6 @@
       LOG(INFO) << "MakeExecutable " << PrettyMethod(method) << " code=" << method_code;
       OatFile::OatMethod oat_method(method_code,
                                     compiled_method->GetFrameSizeInBytes(),
-                                    compiled_method->GetReturnPcOffsetInBytes(),
                                     compiled_method->GetCoreSpillMask(),
                                     compiled_method->GetFpSpillMask(),
                                     &compiled_method->GetMappingTable()[0],
@@ -134,7 +139,6 @@
                                     kStackAlignment,
                                     0,
                                     0,
-                                    0,
                                     NULL,
                                     NULL,
                                     method_invoke_stub);