[llvm] [z/OS] Add z/OS archive reading support (PR #187110)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 13 08:59:38 PDT 2026


================
@@ -104,7 +105,11 @@ ArchiveMemberHeader::ArchiveMemberHeader(const Archive *Parent,
     *Err = createMemberHeaderParseError(this, RawHeaderPtr, Size);
     return;
   }
-  if (ArMemHdr->Terminator[0] != '`' || ArMemHdr->Terminator[1] != '\n') {
+  if ((ArMemHdr->Terminator[0] != '`' || ArMemHdr->Terminator[1] != '\n') &&
----------------
jh7370 wrote:

Correct me if I'm wrong, but won't this code mean that non-z/OS archives will accept the z/OS terminator and vice versa? That doesn't seem correct to me.

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


More information about the llvm-commits mailing list