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

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 29 13:40:36 PST 2020


tejohnson added inline comments.


================
Comment at: llvm/lib/LTO/LTO.cpp:967
+
+  // Ensure we don't have inconsistently split LTO units with type tests.
+  if (Error Err = checkPartiallySplit())
----------------
Can you add a FIXME here noting that this is checking both Regular and Thin LTO and should be split into two, one checking Regular here and the other checking the ThinLTO index in runThinLTO?

This does work since we always go through this path, but logically it would make more sense to do the ThinLTO checking in runThinLTO. Unfortunately, the regular LTO checking this is doing must be after linkRegularLTO.


================
Comment at: llvm/test/LTO/Resolution/X86/dead-strip-fulllto.ll:19
 
+; REMARK: - Function:        dead2
+
----------------
Is the yaml remark more verbose than this (i.e. it includes your message)? Would be good to include more of it.


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

https://reviews.llvm.org/D73597





More information about the llvm-commits mailing list