yaxunl added inline comments. ================ Comment at: test/Frontend/opencl-blocks.cl:9 @@ -8,1 +8,3 @@ // RUN: %clang_cc1 %s -triple amdgcn--amdhsa -x c -std=c99 -verify -fsyntax-only +// RUN: %clang_cc1 -cl-std=CL1.1 -cl-strict-aliasing %s 2>&1 | FileCheck --check-prefix=CHECK-INVALID-OPENCL-VERSION11 %s +// RUN: %clang_cc1 -cl-std=CL1.2 -cl-strict-aliasing %s 2>&1 | FileCheck --check-prefix=CHECK-INVALID-OPENCL-VERSION12 %s ---------------- Anastasia wrote: > yaxunl wrote: > > Better separate these tests to another file, e.g. cl-strict-aliasing.cl > I think it was right to have this in test/Driver/opencl.cl. Why was it moved > in here? This is a frontend option and the warning is emitted by frontend, not the driver. The driver does not consume the option. It just passes it to the frontend.
The driver test is mostly testing whether the option is correctly passed to the compiler invocation by using -###. Repository: rL LLVM http://reviews.llvm.org/D22170 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
