[PATCH] D44786: Lowering x86 adds/addus/subs/subus intrinsics (clang)

[Craig Topper via Phabricator via cfe-commits]

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