[PATCH] D89100: [GlobalISel] Look through truncs and extends in narrowScalarShift

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 9 09:56:43 PDT 2021


paquette accepted this revision.
paquette added a comment.
This revision is now accepted and ready to land.

I think this seems reasonable with a minor nit



================
Comment at: llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp:4883
+    return narrowScalarShiftByConstant(MI, VRegAndVal->Value, HalfTy,
+                                       MRI.getType(Amt));
   }
----------------
Reuse ShiftAmtTy?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89100



More information about the llvm-commits mailing list