jdoerfert added a comment. Can we have a test for this?
================ Comment at: clang/include/clang/CodeGen/BackendUtil.h:48 - void EmbedObject(llvm::Module *M, const CodeGenOptions &CGOpts, DiagnosticsEngine &Diags); ---------------- Seems unrelated, please remove it from this patch. ================ Comment at: clang/lib/CodeGen/BackendUtil.cpp:1774 "could not open '%0' for embedding"); - Diags.Report(DiagID) << std::get<0>(FilenameAndSection); return; ---------------- Same as above? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120270/new/ https://reviews.llvm.org/D120270 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits