JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land.
LGTM with two inline nits. ================ Comment at: clang/lib/Frontend/Rewrite/FrontendActions.cpp:188 std::unique_ptr<raw_ostream> OS = - CI.createDefaultOutputFile(true, getCurrentFileOrBufferName()); + CI.createDefaultOutputFile(false, getCurrentFileOrBufferName()); if (!OS) return; ---------------- ================ Comment at: clang/lib/Frontend/Rewrite/FrontendActions.cpp:273 OutputStream = - CI.createDefaultOutputFile(true, getCurrentFileOrBufferName()); + CI.createDefaultOutputFile(false, getCurrentFileOrBufferName()); if (!OutputStream) ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96363/new/ https://reviews.llvm.org/D96363 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits