[PATCH] D81620: [SVE] Allow scalable vector splats to be gep indices

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 17 17:17:30 PDT 2020


efriedma added a comment.

Posted https://reviews.llvm.org/D82061 to try to make the GEP-related bits more sane.



================
Comment at: llvm/lib/IR/Constants.cpp:1572
 
 const APInt &Constant::getUniqueInteger() const {
   if (const ConstantInt *CI = dyn_cast<ConstantInt>(this))
----------------
Constant::getUniqueInteger() is a terrible API.  I'd prefer to fix the callers so it doesn't need to handle scalable vectors.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81620





More information about the llvm-commits mailing list