[PATCH] D104450: [GlobalISel] Add scalable property to LLT types.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 18 15:11:26 PDT 2021


arsenm added a comment.

LGTM as long as this isn't increasing LLT's size. If it is I would rather steal a bit from the size or elements



================
Comment at: llvm/include/llvm/Support/LowLevelTypeImpl.h:273
   ///   SizeOfElement: 32;
+  ///   Scalable: 1;
   static const constexpr BitFieldInfo VectorElementsFieldInfo{16, 0};
----------------
Is this increasing the total LLT size beyond 64-bits?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104450



More information about the llvm-commits mailing list