[PATCH] D31667: [Sema] Extend GetSignedVectorType to deal with non ExtVector types

Simon Dardis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 6 03:49:10 PDT 2017


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


================
Comment at: lib/Sema/SemaExpr.cpp:9736
+  }
+
+  if (TypeSize == Context.getTypeSize(Context.LongLongTy))
----------------
ahatanak wrote:
> This isn't particularly urgent, but can we use ASTContext ::getIntTypeForBitwidth here rather than calling getVectorType or getExtVectorType in many places?
> 
Yeah, it appears we can. I'll commit the current version and post a refactoring patch.


https://reviews.llvm.org/D31667





More information about the cfe-commits mailing list