arsenm accepted this revision. arsenm added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: test/SemaOpenCL/logical-ops.cl:1-3 +// RUN: %clang_cc1 %s -verify -cl-std=CL1.1 -triple x86_64-unknown-linux-gnu +// RUN: %clang_cc1 %s -verify -cl-std=CL1.2 -triple x86_64-unknown-linux-gnu + ---------------- Anastasia wrote: > Anastasia wrote: > > arsenm wrote: > > > arsenm wrote: > > > > Should this have a 2.0 run line for good measure? > > > 1.0 too I suppose > > Sure! > Apparently CL1.0 is not supported! I missed that.... That sounds like a bug? https://reviews.llvm.org/D29038 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
