[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-07 Thread Anastasia Stulova via cfe-commits
Anastasia added a comment. Cool! Thanks! Repository: rL LLVM https://reviews.llvm.org/D26302 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-07 Thread Yaxun Liu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL286114: [OpenCL] Remove redundant test for OpenCL header file (authored by yaxunl). Changed prior to commit: https://reviews.llvm.org/D26302?vs=76932&id=77039#toc Repository: rL LLVM https://reviews

[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-07 Thread Yaxun Liu via cfe-commits
yaxunl added a comment. In https://reviews.llvm.org/D26302#587480, @Anastasia wrote: > LGTM! Thanks! Do you know the runtime of this test now? It takes about 1s in releaser build and 20s in debug build on my machine. https://reviews.llvm.org/D26302 _

[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-05 Thread Anastasia Stulova via cfe-commits
Anastasia added a comment. LGTM! Thanks! Do you know the runtime of this test now? https://reviews.llvm.org/D26302 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-04 Thread Yaxun Liu via cfe-commits
yaxunl updated this revision to Diff 76932. yaxunl added a comment. Revised as Anastasia suggested. https://reviews.llvm.org/D26302 Files: test/Headers/opencl-c-header.cl Index: test/Headers/opencl-c-header.cl === --- test/Head

[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-04 Thread Anastasia Stulova via cfe-commits
Anastasia added inline comments. Comment at: test/Headers/opencl-c-header.cl:48 -// === -// Compile for OpenCL 1.0 for the first time. A module should be generated. -// RUN: %clang_cc1 -triple spir-unknown-unknown -emit-llvm -o - -finclude-default-header -fmodules -fimplicit-mod

[PATCH] D26302: [OpenCL] Remove redundant test for OpenCL header file

2016-11-04 Thread Yaxun Liu via cfe-commits
yaxunl created this revision. yaxunl added a reviewer: Anastasia. yaxunl added subscribers: cfe-commits, rsmith. https://reviews.llvm.org/D26302 Files: test/Headers/opencl-c-header.cl Index: test/Headers/opencl-c-header.cl === --