[PATCH] D18006: [lto] Add beginning of -save-temps option.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 9 12:03:08 PST 2016


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

We generally don't add much code for debugging, but since the amount of code for this is small, this should be okay.


================
Comment at: ELF/SymbolTable.cpp:150
@@ -149,1 +149,3 @@
 
+static void saveBCFile(std::string Path, Module &M) {
+  std::error_code EC;
----------------
Add a comment that this function is for debugging.

================
Comment at: ELF/SymbolTable.cpp:167
@@ -159,1 +166,3 @@
+  if (Config->SaveTemps)
+    saveBCFile(Config->OutputFile.str() + ".lto.bc", Combined);
   std::unique_ptr<InputFile> F = codegen(Combined);
----------------
Instead of creating a file name here, I'd do that in saveBCFile.


http://reviews.llvm.org/D18006





More information about the llvm-commits mailing list