[PATCH] D150790: [GlobalIsel][X86] fix legalization of G_CTLZ and GTCTPOP

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 09:31:18 PDT 2023


RKSimon added a comment.

In D150790#4350163 <https://reviews.llvm.org/D150790#4350163>, @tschuett wrote:

> That will need larger refactorings. I have already run into this issue. You cannot have two getActionDefinitionsBuilder with the same operation. If I add popcount to setLegalizerInfo32bit, then the builder will be called twice with the same operation: once for 32 and once for 64 bit.

It might be separate setLegalizerInfo32bit  / setLegalizerInfo64bit isn't actually useful


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150790



More information about the llvm-commits mailing list