awarzynski added inline comments.
================
Comment at: flang/lib/Frontend/FrontendActions.cpp:419
+
+ // ... otherwise, print to a file.
+ auto os{ci.CreateDefaultOutputFile(
----------------
kiranchandramohan wrote:
> awarzynski wrote:
> > kiranchandramohan wrote:
> > > Nit: Should we test the existence of such a file?
> > We do :) Sort of!
> >
> > To me, "existence" is a low level concept. Files are handled through e.g.
> > `llvm::raw_fd_ostream` (and occasionally other streams) and IMO all low
> > level details should be dealt with there (and indeed are). `flang-new`
> > should only verify that `os` is not a `nullptr`. If the file does not
> > exist, `os` will be a `nullptr` and that's checked further down. If the
> > file does exist, then everything is fine and we can move to the next step.
> OK, that was sounds fine. I was also meaning to ask whether we should have a
> test that a *.mlir file is generated in a test.
I can add one, yes!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118985/new/
https://reviews.llvm.org/D118985
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
