[PATCH] D123211: [flang][driver] Add support for generating LLVM bytecode files

[Emil Kieri via Phabricator via cfe-commits]

ekieri accepted this revision.
ekieri added a comment.

One nit/question inline, otherwise LGTM.



================
Comment at: flang/lib/Frontend/FrontendActions.cpp:491
+  // Create and configure `TargetMachine`
+  std::unique_ptr<llvm::TargetMachine> TM;
+
----------------
Is there a reason why use TM.reset instead of initialising TM like you do with os below?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123211