[PATCH] D54253: [OpenCL][NFC] Improve test coverage of test/Index/opencl-types.cl

Anastasia Stulova via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 16 07:43:51 PST 2018


Anastasia added a comment.

In https://reviews.llvm.org/D54253#1300827, @AlexeySachkov wrote:

> I'm running OpenCL on an x886 and everything is fine, but there are a lot of build bots which build different targets on different architectures. Since there are no `target` option specified, `c-index-test` uses native target.
>
> I just realized: may be it is better to add something like `// REQUIRES: x86` to the test?


Yes, it's the same for clang actually. We are solving this by adding `-triple` explicitly, alternatively `REQUIRES` will work too.


Repository:
  rC Clang

https://reviews.llvm.org/D54253





More information about the cfe-commits mailing list