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

Peter Rong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 2 13:29:26 PST 2023


Peter added a comment.

In D140747#4019289 <https://reviews.llvm.org/D140747#4019289>, @nikic wrote:

> Why do some parts of this use APSInt?

The old code assumes the branch case to be signed integer (`int64_t`), therefore some place use `APSInt` to make signess more clear.


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