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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 06:12:45 PST 2023


RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LowerSwitch.cpp:461
+      APInt N = High - Low + 1;
+      assert(N.sge(SignedZero));
+      // Explict insert to make sure the bitwidth of APInts match
----------------
RKSimon wrote:
> (style) assert message
It looks like this can't handle i1 types properly


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