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

Peter Klausler via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 25 13:28:59 PDT 2022


klausler added a comment.

Instead of adding new custom APIs that let command-line options control behavior is a way that is redundant with the runtime environment, I suggest that you try a more general runtime library API by which the main program can specify a default environment variable setting, or a set of them.  Then turn the command-line options into the equivalent environment settings and pass them as default settings that could be overridden by the actual environment.

This would not be any more work, it would lead to a cleaner implementation in the runtime than this one, and it would make the next option of this kind much easier to implement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130513



More information about the cfe-commits mailing list