[PATCH] D151803: [GlobalIsel][X86] Legalize G_BSWAP II

[Thorsten via Phabricator via llvm-commits]

tschuett updated this revision to Diff 527492.
tschuett added a comment.

add s17


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151803

Files:
  llvm/lib/Target/X86/X86LegalizerInfo.cpp
  llvm/test/CodeGen/X86/GlobalISel/legalize-bswap.mir


Index: llvm/test/CodeGen/X86/GlobalISel/legalize-bswap.mir
===================================================================
--- llvm/test/CodeGen/X86/GlobalISel/legalize-bswap.mir
+++ llvm/test/CodeGen/X86/GlobalISel/legalize-bswap.mir
@@ -2,7 +2,7 @@
 # RUN: llc -mtriple=x86_64-linux-gnu -run-pass=legalizer %s -o - | FileCheck %s --check-prefix=X86-64
 # RUN: llc -mtriple=i386-linux-gnu -run-pass=legalizer %s -o - | FileCheck %s --check-prefix=X86-32
 
-# test BSWAP s16, s32, and s64
+# test BSWAP s16, s17, s32, and s64
 
 ...
 ---
Index: llvm/lib/Target/X86/X86LegalizerInfo.cpp
===================================================================
--- llvm/lib/Target/X86/X86LegalizerInfo.cpp
+++ llvm/lib/Target/X86/X86LegalizerInfo.cpp
@@ -105,7 +105,7 @@
       .clampScalar(0, s32, s64);
 
     if (Subtarget.hasPOPCNT()) {
-      // popcount
+      // popcount (POPCNT)
       getActionDefinitionsBuilder(G_CTPOP)
         .legalFor({{s16, s16}, {s32, s32}, {s64, s64}})
         .widenScalarToNextPow2(1, /*Min=*/16)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D151803.527492.patch
Type: text/x-patch
Size: 1033 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230601/0af31f40/attachment.bin>