[lldb] [PECOFF] Only use PECallFrameInfo on the one supported architecture

mstorsjo · tstellar · commit aba4e3fa3bd0 · 2020-05-18T18:24:44.000-07:00
The RuntimeFunction struct, which PECallFrameInfo interprets, has a different layout and differnet semantics on all architectures. Differential Revision: https://reviews.llvm.org/D77000 (cherry picked from commit aa786b8)
diff --git a/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp b/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
@@ -780,6 +780,9 @@ std::unique_ptr<CallFrameInfo> ObjectFilePECOFF::CreateCallFrameInfo() {
   if (!data_dir_exception.vmaddr)
     return {};
 
+  if (m_coff_header.machine != llvm::COFF::IMAGE_FILE_MACHINE_AMD64)
+    return {};
+
   return std::make_unique<PECallFrameInfo>(*this, data_dir_exception.vmaddr,
                                            data_dir_exception.vmsize);
 }

Original file line number	Diff line number	Diff line change
`@@ -780,6 +780,9 @@ std::unique_ptr<CallFrameInfo> ObjectFilePECOFF::CreateCallFrameInfo() {`
`780`	`780`	`if (!data_dir_exception.vmaddr)`
`781`	`781`	`return {};`
`782`	`782`
	`783`	`+ if (m_coff_header.machine != llvm::COFF::IMAGE_FILE_MACHINE_AMD64)`
	`784`	`+ return {};`
	`785`	`+`
`783`	`786`	`return std::make_unique<PECallFrameInfo>(*this, data_dir_exception.vmaddr,`
`784`	`787`	`data_dir_exception.vmsize);`
`785`	`788`	`}`