[Lldb-commits] [lldb] [lldb][AIX] Header Parsing for XCOFF Object File in AIX (PR #116338)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 9 07:14:05 PST 2024


================
@@ -136,13 +169,42 @@ bool ObjectFileXCOFF::MagicBytesMatch(DataBufferSP &data_sp,
   return XCOFFHeaderSizeFromMagic(magic) != 0;
 }
 
-bool ObjectFileXCOFF::ParseHeader() { return false; }
+bool ObjectFileXCOFF::ParseHeader() {
+
+  bool retVal = false;
+  ModuleSP module_sp(GetModule());
+  if (module_sp) {
+    std::lock_guard<std::recursive_mutex> guard(module_sp->GetMutex());
----------------
labath wrote:

Given that this is called directly from CreateInstance, I'm pretty sure this locking is not needed (ObjectFileELF definitely doesn't do that, although PECOFF for some reason does).

https://github.com/llvm/llvm-project/pull/116338


More information about the lldb-commits mailing list