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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
