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
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to