[PATCH] D150755: [GlobalIsel][x86] Legalize G_CTPOP and G_CTLZ II

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 07:56:37 PDT 2023


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86LegalizerInfo.cpp:394
+  const LLT s32 = LLT::scalar(32);
+  const LLT s64 = LLT::scalar(64);
+
----------------
Additionally, don't we need to guard against 32-bit triples where s64 isn't legal?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150755



More information about the llvm-commits mailing list