craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

LGTM with that one comment.



================
Comment at: lib/CodeGen/CGBuiltin.cpp:8288
+    llvm::Type *ElementType = ResultType->getVectorElementType();
+    llvm::Type *ExtElementType = ElementType == CGF.Builder.getInt8Ty()
+                                     ? CGF.Builder.getInt16Ty()
----------------
You could probably just do ResultType->getScalarSizeInBits() == 8


Repository:
  rC Clang

https://reviews.llvm.org/D44786



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to