[PATCH] D83948: [PowerPC] Custom lowering for funnel shifts

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 01:19:26 PDT 2020


RKSimon added a comment.

Have you triaged whats going on in the mulfixsat test regressions?



================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:8998
+  // fshr: (X << (BW - (Z % BW))) | (Y >> (Z % BW))
+  // This is simpler than TargetLowering::expandFunnelShift because we can rely
+  // on PowerPC shift by BW being well defined.
----------------
Maybe also show the pseudocode as used by PPC for comparison?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83948





More information about the llvm-commits mailing list