[PATCH] D138986: [AIX][BigArchive] Treat the archive is empty if the first child member offset is zero

Esme Yi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 08:40:12 PST 2022


Esme accepted this revision.
Esme added a comment.
This revision is now accepted and ready to land.

LGTM. Thank you for fixing it.



================
Comment at: llvm/include/llvm/Object/Archive.h:416
   uint64_t getLastChildOffset() const { return LastChildOffset; }
-  bool isEmpty() const override {
-    return Data.getBufferSize() == sizeof(FixLenHdr);
-  };
+  bool isEmpty() const override { getFirstChildOffset() == 0; };
 };
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138986



More information about the llvm-commits mailing list