r190129 - OpenCL allows the (pre/post)-(increment/decrement) operator on integer vector types,

David Tweed david.tweed at arm.com
Fri Sep 6 02:58:09 PDT 2013


Author: davidtweed
Date: Fri Sep  6 04:58:08 2013
New Revision: 190129

URL: http://llvm.org/viewvc/llvm-project?rev=190129&view=rev
Log:
OpenCL allows the (pre/post)-(increment/decrement) operator on integer vector types,
so allow that case and add appropriate tests.

Patch by Ruiling Song!


Added:
    cfe/trunk/test/SemaOpenCL/vector_inc_dec_ops.cl
Modified:
    cfe/trunk/lib/Sema/SemaExpr.cpp

Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=190129&r1=190128&r2=190129&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExpr.cpp Fri Sep  6 04:58:08 2013
@@ -8413,6 +8413,9 @@ static QualType CheckIncrementDecrementO
                                           IsInc, IsPrefix);
   } else if (S.getLangOpts().AltiVec && ResType->isVectorType()) {
     // OK! ( C/C++ Language Extensions for CBEA(Version 2.6) 10.3 )
+  } else if(S.getLangOpts().OpenCL && ResType->isVectorType() &&
+            ResType->getAs<VectorType>()->getElementType()->isIntegerType()) {
+    // OpenCL V1.2 6.3 says dec/inc ops operate on integer vector types.
   } else {
     S.Diag(OpLoc, diag::err_typecheck_illegal_increment_decrement)
       << ResType << int(IsInc) << Op->getSourceRange();

Added: cfe/trunk/test/SemaOpenCL/vector_inc_dec_ops.cl
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaOpenCL/vector_inc_dec_ops.cl?rev=190129&view=auto
==============================================================================
--- cfe/trunk/test/SemaOpenCL/vector_inc_dec_ops.cl (added)
+++ cfe/trunk/test/SemaOpenCL/vector_inc_dec_ops.cl Fri Sep  6 04:58:08 2013
@@ -0,0 +1,19 @@
+// RUN: %clang_cc1 %s -verify -pedantic -fsyntax-only
+// expected-no-diagnostics
+
+typedef __attribute__((ext_vector_type(2)))  char char2;
+typedef __attribute__((ext_vector_type(4)))  unsigned int uint4;
+typedef __attribute__((ext_vector_type(8)))  long long8;
+
+void vectorIncrementDecrementOps()
+{
+  char2 A = (char2)(1);
+  uint4 B = (uint4)(1);
+  long8 C = (long8)(1);
+
+  A++;
+  --A;
+  B--;
+  ++B;
+  C++;
+}





More information about the cfe-commits mailing list