[PATCH] D111889: [AIX] Support of Big archive (read)

Digger Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 17 06:09:31 PST 2022


DiggerLin marked 8 inline comments as done.
DiggerLin added inline comments.


================
Comment at: llvm/lib/Object/Archive.cpp:476
 
-  // Check to see if this is past the end of the archive.
   if (NextLoc > Parent->Data.getBufferEnd()) {
----------------
jhenderson wrote:
> Let's readd this comment (sorry if I asked for it to be removed earlier...)
the code


>   std::string Msg("offset to next archive member past the end of the archive  after member ");

explain it.

but I added it anyway.





Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111889



More information about the llvm-commits mailing list