[PATCH] D81495: [SVE] Remove invalid calls to VectorType::getNumElements from BasicTTIImpl
    Christopher Tetreault via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Jun  9 13:48:38 PDT 2020
    
    
  
ctetreau added a comment.
> I'm not sure the `FIXME_ScalableVectorNotSupported` thing is useful here, as opposed to just writing `cast<FixedVectorType>()`.
I like it for the reasons I mentioned. Strictly speaking, it is equivalent to just doing the cast, but I think it provides documentation value. If people feel strongly about removing it, I can do so.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81495/new/
https://reviews.llvm.org/D81495
    
    
More information about the llvm-commits
mailing list