[PATCH] D97785: [SystemZ][z/OS] Distinguish between text and binary files on z/OS

Abhina Sree via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 11:42:26 PDT 2021


abhina.sreeskantharajan added a comment.

In D97785#2651008 <https://reviews.llvm.org/D97785#2651008>, @amccarth wrote:

> When changing an IO stream's mode from binary to text breaks only Windows, it's likely due to the fact that Windows uses CR+LF as the newline indicator.  Not only are the CRs sometimes unexpected, they can also throw off code that tries to seek to a computed file position.

Yes, this is probably the cause. I created a patch to set Binary mode on Windows only https://reviews.llvm.org/D99363 which should workaround the issue and unblock the bot. I do plan to investigate a better fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97785



More information about the llvm-commits mailing list