[PATCH] D73597: [LTO] Add optimization remarks for removed functions

Francis Visoiu Mistrih via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 28 17:47:02 PST 2020


thegameg added inline comments.


================
Comment at: llvm/lib/LTO/LTO.cpp:790
+        auto RemStr =
+            formatv("Function not added to combined module: {0}", F->getName());
+        ORE.emit(OptimizationRemark(DEBUG_TYPE, "deadfunction", F)
----------------
This is not the usual way we add strings with arguments to remarks. Please use `<< "STR: " << ore::NV("Function", F)`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73597/new/

https://reviews.llvm.org/D73597





More information about the llvm-commits mailing list