[PATCH] D140747: [Transform] Rewrite LowerSwitch using APInt

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 01:06:43 PST 2023


uabelho added a comment.
Herald added a subscriber: StephenFan.

Hello,

The following starts crashing with this patch:

  opt -passes=lowerswitch bbi-77637.ll -o /dev/null

It fails with

  opt: ../lib/Transforms/Utils/LowerSwitch.cpp:461: void (anonymous namespace)::ProcessSwitchInst(llvm::SwitchInst *, SmallPtrSetImpl<llvm::BasicBlock *> &, llvm::AssumptionCache *, llvm::LazyValueInfo *): Assertion `N.sge(SignedZero) && "Popularity shouldn't be negative."' failed.

F26157551: bbi-77637.ll <https://reviews.llvm.org/F26157551>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140747



More information about the llvm-commits mailing list