[PATCH] D135022: [GlobalISel] Implement undef combines for funnel shifts

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 1 15:42:14 PDT 2022


arsenm added a comment.

Doesn't remove whatever was incorrectly folding this before



================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/CombinerHelper.h:763-764
+  /// fshr (x, y, undef) -> y for scalars
+  /// fshl (x, y, undef) -> undef for vectors
+  /// fshr (x, y, undef) -> undef for vectors
+  void applyUndefFunnelShiftAmount(MachineInstr &MI);
----------------
Different behavior for vector and scalar sounds really broken


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

https://reviews.llvm.org/D135022



More information about the llvm-commits mailing list