[PATCH] D81573: [CodeGen,AArch64] Fix up warnings in performExtendCombine

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 18 03:14:30 PDT 2020


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

LGTM



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:11841
   unsigned SrcEltSize = SrcVT.getScalarSizeInBits();
-  unsigned ElementCount = SrcVT.getVectorNumElements();
+  ElementCount ElementCount = SrcVT.getVectorElementCount();
   SrcVT = MVT::getVectorVT(MVT::getIntegerVT(SrcEltSize * 2), ElementCount);
----------------
nit: I think it might be worth changing the name here, as "ElementCount ElementCount" looks a bit confusing


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81573





More information about the llvm-commits mailing list