[PATCH] D150677: [GlobalIsel][X86] Legalize G_CTPOP and G_CTLZ

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86LegalizerInfo.cpp:403
+  getActionDefinitionsBuilder(G_CTPOP}
+    .legalFor({s16, s32, s64})
+    .widenScalarToNextPow2(1, /*Min=*/16)
----------------
arsenm wrote:
> These both have buggy type lists, and I'm surprised nothing is blowing up. These have 2 type parameters, the result may be different from the source 



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150677