[Lldb-commits] [lldb] [lldb][AIX] Taking Linux Host Info header's base for AIX (PR #106910)

Dhruv Srivastava via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 2 01:42:25 PDT 2024


DhruvSrivastavaX wrote:

Thank you for your feedback; I understand the concern about potential code duplication. Indeed, components like `PlatformAIX, HostInfoAIX, and NativeProcess/Thread` do share significant similarities with their Linux counterparts. However, several other components, such as `ObjectFileXCOFF, the Big Archive container, and AIX-DYLD, `require more substantial changes due to platform-specific aspects like the AIX base object file formats, hardware architecture (PPC64), and other inherent differences.

We've also had to exclude certain functionalities that aren't feasible on AIX at this stage. The main intentionn for our current approach is to isolate platform dependencies, thereby avoiding conditional directives like `#if AIX #else` within the common codebase. Even in the draft PR, our long-term goal is to minimize such conditionals wherever possible.

That said, we've made every effort to keep the code as generic as possible, and we’re open to any further suggestions on how to improve this. It will help in refining the changes.

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


More information about the lldb-commits mailing list