[PATCH] D87814: [AArch64][GlobalISel] Fix bug in fewVectorElts action while legalizing oversize G_FPTRUNC vectors

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 07:23:47 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp:3280
   LLT NarrowTy1;
   unsigned NumParts;
 
----------------
I've been meaning to delete this function since it should now be redundant


================
Comment at: llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp:3289
+    NarrowTy1 = LLT::vector(NarrowTy.getNumElements(),
+                            SrcTy.getElementType().getSizeInBits());
   } else {
----------------
Just .getElementType


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87814



More information about the llvm-commits mailing list