[PATCH] D26886: [ThinLTO] Implement -pass-remarks-output in ThinLTOCodeGenerator

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 10 12:42:03 PST 2017


davide added inline comments.


================
Comment at: llvm/trunk/lib/LTO/ThinLTOCodeGenerator.cpp:858-863
+        auto DiagFileOrErr = setupOptimizationRemarks(Context, count);
+        if (!DiagFileOrErr) {
+          errs() << "Error: " << toString(DiagFileOrErr.takeError()) << "\n";
+          report_fatal_error("ThinLTO: Can't get an output file for the "
+                             "remarks");
+        }
----------------
I'm working on refactoring this so we can share between old and new LTO API. I noticed that if you comment out this block of code, the test still passes. Any ideas on how we can better test this?


Repository:
  rL LLVM

https://reviews.llvm.org/D26886





More information about the llvm-commits mailing list