commit d9efea646485fe996c967f4680a3fc4cdc40b251
author Andreas Gampe <agampe@google.com> Mon Jul 21 22:56:08 2014 -0700
committer Andreas Gampe <agampe@google.com> Tue Aug 26 10:54:56 2014 -0700
tree 3fd198f4c7a21e34d1130d5bcda20d892fb21b07
parent e18aa4316eb9a15cd6b1051f27a1ce49967c170e

ART: Improve error message on oat writer failure

Try to log the pending exception to help resolve issues.

Bug: 16406811
Change-Id: I035ae9e59a5ee02b9e90c35c0644ec088d3c7d12

compiler/oat_writer.cc

diff --git a/compiler/oat_writer.cc b/compiler/oat_writer.cc
index 41a91ed..680ce0a 100644
--- a/compiler/oat_writer.cc
+++ b/compiler/oat_writer.cc
@@ -527,7 +527,14 @@
                                                       NullHandle<mirror::ClassLoader>(),
                                                       NullHandle<mirror::ArtMethod>(),
                                                       invoke_type);
-    CHECK(method != NULL) << PrettyMethod(it.GetMemberIndex(), *dex_file_, true);
+    if (method == nullptr) {
+      LOG(ERROR) << "Unexpected failure to resolve a method: "
+                 << PrettyMethod(it.GetMemberIndex(), *dex_file_, true);
+      soa.Self()->AssertPendingException();
+      mirror::Throwable* exc = soa.Self()->GetException(nullptr);
+      std::string dump = exc->Dump();
+      LOG(FATAL) << dump;
+    }
     // Portable code offsets are set by ElfWriterMclinker::FixupCompiledCodeOffset after linking.
     method->SetQuickOatCodeOffset(offsets.code_offset_);
     method->SetOatNativeGcMapOffset(offsets.gc_map_offset_);

runtime/thread.cc

diff --git a/runtime/thread.cc b/runtime/thread.cc
index 7ac685b..0eaac6b 100644
--- a/runtime/thread.cc
+++ b/runtime/thread.cc
@@ -1101,6 +1101,12 @@
       (*tlsPtr_.name == kThreadNameDuringStartup);
 }
 
+void Thread::AssertPendingException() const {
+  if (UNLIKELY(!IsExceptionPending())) {
+    LOG(FATAL) << "Pending exception expected.";
+  }
+}
+
 void Thread::AssertNoPendingException() const {
   if (UNLIKELY(IsExceptionPending())) {
     ScopedObjectAccess soa(Thread::Current());

runtime/thread.h

diff --git a/runtime/thread.h b/runtime/thread.h
index fe950c4..5283ca6 100644
--- a/runtime/thread.h
+++ b/runtime/thread.h
@@ -321,6 +321,7 @@
     return tlsPtr_.exception;
   }
 
+  void AssertPendingException() const;
   void AssertNoPendingException() const;
   void AssertNoPendingExceptionForNewException(const char* msg) const;