[PATCH] D133706: ControlHeightReduction: Remove assert check in shouldApply

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 9 06:25:07 PDT 2022


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

LGTM

We also hit this assert here: https://github.com/rust-lang/rust/pull/101403#issuecomment-1272519304 I believe in that case the problem is that some of the modules participating in ThinLTO do not have PGO information (and that's fine).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133706



More information about the llvm-commits mailing list