[PATCH] D152445: [GlobalIsel][X86] Legalize G_SELECT

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 09:31:08 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86LegalizerInfo.cpp:313
+  getActionDefinitionsBuilder(G_SELECT)
+    .legalFor({s8, s16, s32, s64, p0})
+    .widenScalarToNextPow2(0, /*Min=*/8)
----------------
 select has two type indexes. I'm surprised this isn't asserting. You need to specify these as pairs with the condition type 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152445



More information about the llvm-commits mailing list