[PATCH] D49434: Put "built-in" function definitions in global Used list, for LTO. (fix bug 34169)

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

You wouldn't be making changes to the IR, only the summary. I was thinking that you might add code to http://llvm-cs.pcc.me.uk/lib/Analysis/ModuleSummaryAnalysis.cpp#534 that would effectively do:

  if (ValueInfo VI = Index.getValueInfo(GlobalValue::getGUID("memcpy")))
      for (auto &Summary : VI.getSummaryList())
        Summary->setLive(true);
  if (ValueInfo VI = Index.getValueInfo(GlobalValue::getGUID("memmove")))
      for (auto &Summary : VI.getSummaryList())
        Summary->setLive(true);
  // etc.


https://reviews.llvm.org/D49434