[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
Thu Dec 8 00:40:00 PST 2022


Esme added inline comments.


================
Comment at: llvm/include/llvm/Object/Archive.h:418
+    return Data.getBufferSize() == sizeof(FixLenHdr) ||
+           getFirstChildOffset() == 0;
   };
----------------
Can we remove the check of `Data.getBufferSize() == sizeof(FixLenHdr)`? I think `getFirstChildOffset() == 0` can cover that case.


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