[PATCH] D77587: [SVE] Add new VectorType subclasses

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 7 16:55:05 PDT 2020


ctetreau marked an inline comment as done.
ctetreau added inline comments.


================
Comment at: llvm/include/llvm/IR/Type.h:76
+    PointerTyID,       ///< 15: Pointers
+    VectorTyID,        ///< 16: SIMD 'packed' format, or other vector type
+    FixedVectorTyID,   ///< 17: Fixed width SIMD vector type
----------------
efriedma wrote:
> ctetreau wrote:
> > efriedma wrote:
> > > efriedma wrote:
> > > > VectorTyID should be dead.
> > > Oh, there's still a bunch of uses of VectorTyID.  You probably need to take care of that in this patch.
> > Since this thing is exposed from the C api, it might break external code to remove it.
> > 
> > However, it is the design that it should never be possible to construct a base VectorType. Maybe it makes sense to delete it here, but leave the type in the C api?
> Whether or not we actually remove LLVMVectorTypeKind from the enum, it's a breaking change from the C API's perspective if vector types don't have the right ID.  So we have two options: either make the breaking change to the C API, or don't expose the Scalable/Fixed split via the C API at all (and explicitly remap the internal IDs in LLVMGetTypeKind).
> 
> Either way, that isn't a reason to keep around the C++ VectorTyID; we don't expose that directly anyway.
Fair enough, I'll get rid of it


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77587





More information about the llvm-commits mailing list