[PATCH] D144872: [AIX] Align the content of an xcoff object file which has auxiliary header in big archive.

[Digger Lin via Phabricator via llvm-commits]

DiggerLin added a comment.

In D144872#4579160 <https://reviews.llvm.org/D144872#4579160>, @jhenderson wrote:

> I'll be honest, I'm not convinced the test coverage looks to be anywhere near comprehensive enough. However, I also haven't attempted to review it versus the code changes to check how much of the new code is actually covered. I'd suggest you consider using a code coverage tool to see how much coverage there is of your new code.



  In the test big-archive-xcoff-auxi-head-align.test , it test whether the XCOFF object data member which has auxiliary header  align correctly  base on the information of  auxiliary header.  I think it is enough here.  in AIX OS since build bot use  CMake 3.22, it use llvm-ar instead of  AIX OS ar.  it will test the functionality too. (we have to add the -DCMAKE_AR=/usr/bin/ar when we compile, since llvm-ar do not align XCOFF object correctly for big archive in AIX os ). after the the patch commit, we will use the `llvm-ar` instead of  the AIX `ar`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144872