[flang-commits] [PATCH] D106727: [flang] Produce proper "preprocessor output" for -E option

[Peter Klausler via Phabricator via flang-commits]

klausler marked 4 inline comments as done.
klausler added inline comments.


================
Comment at: flang/include/flang/Parser/parsing.h:40
   Fortran::parser::Encoding encoding{Fortran::parser::Encoding::UTF_8};
+  bool prescanOnly{false}; // -E
 };
----------------
awarzynski wrote:
> klausler wrote:
> > awarzynski wrote:
> > > [nit] Why not preprocess only? This is triggered by `-E` and does a bit more than just prescanning. But I don't mind.
> > From the perspective of the Parsing module, we're just going to run the prescanner.  F18 has no preprocessing phase; there is no way to just run its preprocessor.  If there's a better accurate name that you would prefer, perhaps "preprocessedOutputOnly", let me know.
> `prescanAndReformat`?
Whatever you like; will do.


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

https://reviews.llvm.org/D106727