[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