[PATCH] D147133: [IPO] Add documentation for new function argument

[Teresa Johnson via Phabricator via llvm-commits]

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

Thanks! lgtm with minor wording tweak suggested



================
Comment at: llvm/include/llvm/Transforms/IPO/FunctionImport.h:139
 ///
+/// \p isPrevailing is a function-like object that will be called with a global
+/// value's GUID and summary and should return whether the module corresponding
----------------
Maybe just "callback" instead of "function-like object"? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147133