This is an archive of the discontinued LLVM Phabricator instance.

[OpenCL] Actually activate Frontend/opencl.cl test and fix test bugs
ClosedPublic

Authored by mboehme on Jul 14 2016, 5:51 AM.

Details

Summary

rL275318 added the test Frontend/opencl.cl test, but that test was never actually run because Frontend/lit.local.cfg doesn't contain the '.cl' file suffix.

Once the test is activated, it fails with (unintended) compile errors in the newly added CHECK_INVALID_OPENCL_VERSION checks.

This patch adds the '.cl' file suffix to Frontend/lit.local.cfg to activate the test and fixes the test bug by adding '-fblocks' to the relevant command lines.

Diff Detail

Repository
rL LLVM

Event Timeline

mboehme updated this revision to Diff 63959.Jul 14 2016, 5:51 AM
mboehme retitled this revision from to [OpenCL] Actually activate Frontend/opencl.cl test and fix test bugs.
mboehme updated this object.
mboehme added a reviewer: bkramer.
mboehme added subscribers: cfe-commits, ashi1.
bkramer accepted this revision.Jul 14 2016, 6:02 AM
bkramer edited edge metadata.
This revision is now accepted and ready to land.Jul 14 2016, 6:02 AM
This revision was automatically updated to reflect the committed changes.