[PATCH] D71678: [ConstantHoisting] Ignore unreachable bb:s when collecting candidates

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope marked an inline comment as done.
bjope added inline comments.


================
Comment at: llvm/test/Transforms/ConstantHoisting/X86/pr43903-not-all-uses-rebased.ll:4
+
+; This is a reproducer for BBI-35111 / PR43903 where we hit an assertion:
+; opt: ../lib/Transforms/Scalar/ConstantHoisting.cpp:903: bool llvm::ConstantHoistingPass::emitBaseConstants(llvm::GlobalVariable *): Assertion `UsesNum == (ReBasesNum + NotRebasedNum) && "Not all uses are rebased"' failed.
----------------
I'll remove the internal ticket number BBI-355111 before landing this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71678