[PATCH] D95460: [flang][driver] Add forced form flags and -ffixed-line-length
Faris via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 28 05:49:46 PST 2021
FarisRehman marked an inline comment as done.
FarisRehman added inline comments.
================
Comment at: flang/lib/Frontend/CompilerInstance.cpp:151-158
+ if (invoc.frontendOpts().fortranForm_ == FortranForm::Unknown) {
+ // Switch between fixed and free form format based on the input file
+ // extension. Ideally we should have all Fortran options set before
+ // entering this loop (i.e. processing any input files). However, we
+ // can't decide between fixed and free form based on the file extension
+ // earlier than this.
+ invoc.fortranOpts().isFixedForm = fif.IsFixedForm();
----------------
awarzynski wrote:
> Hm, unwanted TABs?
>
> Also, please keep note of: https://reviews.llvm.org/D95464 (there should be no conflicts from what I can tell).
I believe this is just Phabricator showing the comments have been indented
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95460/new/
https://reviews.llvm.org/D95460
More information about the cfe-commits
mailing list