[PATCH] D151459: [GlobalIsel][X86] Legalize G_CTLZ and G_CTPOP for 32-bit

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 10:07:14 PDT 2023


tschuett added a comment.

The underlying issue is that I can legalize G_CTPOP only once. There is VPOPCNTDQ and it has to be in the same builder! With the current setup, they legalize operations for different features. The famous `setLegalizerInfo*`functions.

If I invoke `setLegalizerInfo32bit()` and later `setLegalizerInfoAVX512()`, then G_CTPOP could be legalized twice, which is forbidden!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151459



More information about the llvm-commits mailing list