[PATCH] D146099: [Pipeline] Remove early InstCombine in ThinLTO post link sample profile pipeline

[David Li via Phabricator via llvm-commits]

davidxl accepted this revision.
davidxl added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:976
-  // https://research.google.com/pubs/pub45290.html
-  // FIXME: revisit how SampleProfileLoad/Inliner/ICP is structured.
-  if (LoadSampleProfile)
----------------
tejohnson wrote:
> @davidxl is this comment still needed?
this comment is obsolete as bitcast is no longer there (to block exposing direct calls).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146099