[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
Wed Sep 4 08:16:49 PDT 2024


DhruvSrivastavaX wrote:

Ok sure. 

For readability and clarity, for the files having comparatively bigger changes, I can drop it as multiple commits in the same PR, so that the changes can be seen as cleanly as possible:
1. Commit 1 - copy paste the original
2. Commit 2 - Make AIX dependant changes

And for each of these bigger files, we can have respective PRs as per your suggestion.

And for the rest of the common files, another draft PR should be ok for the readability pov.
It can be used as reference for the changes that will be dropped in future. 

One question though? For the comparatively smaller changes across multiple files, is it okay to club the in one PR if they are related? 

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


More information about the lldb-commits mailing list