[Lldb-commits] [PATCH] D127048: [lldb] Set COFF module ABI from default triple and make it an option

[Muhammad Omair Javaid via Phabricator via lldb-commits]

omjavaid added a comment.

I am not fully familiar with PE/COFF but lets try to get this merged. For starters I have a couple of questions here:

1. This rev apparently assumes x86_64 only but what about Arm/AArch64?
2. If we have DLLs what abi setting they ll follow the one set by the user or default which may be MSVC?
3. Is there anything in PE/COFF header information that can serve is a good guess for the underlying ABI.
4. What more information can we get from debug info in this case? DWARF may not be a good guess as we can generate it for MSVC abi as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127048/new/

https://reviews.llvm.org/D127048