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

mstorsjo · mstorsjo · commit aa786b881fc8 · 2020-04-01T12:39:21.000+03:00
The RuntimeFunction struct, which PECallFrameInfo interprets, has a different layout and differnet semantics on all architectures. Differential Revision: https://reviews.llvm.org/D77000
diff --git a/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp b/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
@@ -782,6 +782,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
`@@ -782,6 +782,9 @@ std::unique_ptr<CallFrameInfo> ObjectFilePECOFF::CreateCallFrameInfo() {`
`782`	`782`	`if (!data_dir_exception.vmaddr)`
`783`	`783`	`return {};`
`784`	`784`
	`785`	`+ if (m_coff_header.machine != llvm::COFF::IMAGE_FILE_MACHINE_AMD64)`
	`786`	`+ return {};`
	`787`	`+`
`785`	`788`	`return std::make_unique<PECallFrameInfo>(*this, data_dir_exception.vmaddr,`
`786`	`789`	`data_dir_exception.vmsize);`
`787`	`790`	`}`