[PATCH] D104451: [GlobalISel] NFC: Change LLT::vector to take ElementCount.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 21 08:22:45 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/include/llvm/Support/LowLevelTypeImpl.h:100
   static LLT scalarOrVector(uint16_t NumElements, LLT ScalarTy) {
-    return NumElements == 1 ? ScalarTy : LLT::vector(NumElements, ScalarTy);
+    // FIXME: Migrate interface to use ElementCount
+    return NumElements == 1 ? ScalarTy
----------------
Is there actually a context where you would use this which would need a scalable vector?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104451



More information about the llvm-commits mailing list