[PATCH] D130513: [Flang] Add -fconvert option to swap endianness for unformatted files

Peixin Qiao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 21 02:36:42 PDT 2022


peixin added a comment.

LGTM. Tested on one little-endian machine with our internal tests (I attached them in the issue https://github.com/llvm/llvm-project/issues/55961) for the priority of `-fconvert=` option and `CONVERT` argument in open statement and all passed. I didn't see the the `GFORTRAN_CONVERT_UNIT` used in HPC workloads and don't have tests for it. For now, I would avoid full combination tests for it.

@jeanPerier @clementval Can you help double check the lowering part?


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

https://reviews.llvm.org/D130513



More information about the cfe-commits mailing list