Hi pocl-devs, seems the clang developers do not like the idea of enabling OpenCL-C pragmas per default for every target, which now left all but x86 (I think) broken for LLVM 3.9.
Below is the forwarded suggestions from the clang devs. A worthy idea to consider is 'pocl' in the target triple. When clang detects this, it could legitimately enable all OpenCL-C pragmas (or do whatever else we deem necessary). All sorts of things could be easier with this. I don't think I have enough time to commit to this. Any takers? kalle -------- Forwarded Message -------- Subject: RE: PATCH: re-enable OpenCL extensions Date: Thu, 19 Jan 2017 21:43:26 +0000 From: Liu, Yaxun (Sam) <[email protected]> To: Anastasia Stulova <[email protected]>, Kalle Raiskila <[email protected]> CC: [email protected] <[email protected]>, nd <[email protected]> I think the supported extensions for a target should be as accurate as possible, for it to be useful. Setting all extensions to be supported on all targets will defeat its purpose. I recommend to introduce "pocl" as an environment in the triple and add supported OpenCL extensions for different targets based on that. Sam -----Original Message----- From: Anastasia Stulova [mailto:[email protected]] Sent: Thursday, January 19, 2017 12:31 PM To: Kalle Raiskila <[email protected]> Cc: [email protected]; Liu, Yaxun (Sam) <[email protected]>; nd <[email protected]> Subject: RE: PATCH: re-enable OpenCL extensions As mentioned on cfe-dev as well, although it doesn't seem too critical it is generally not logical to enable all extensions by default because most of the targets don't even support OpenCL. But I understand your situation with using x86 or ARM backends in a generic way. Do you think this can be solved instead with the new " -cl-ext=" option: http://clang.llvm.org/docs/UsersManual.html#opencl-specific-options May be Sam could comment more since he has done most work with the extensions lately. Cheers, Anastasia -----Original Message----- From: cfe-commits [mailto:[email protected]] On Behalf Of Kalle Raiskila via cfe-commits Sent: 19 January 2017 08:03 To: [email protected] Subject: PATCH: re-enable OpenCL extensions Hi, I noticed a change from clang 3.8 to 3.9, that it disabled all OpenCL extension pragmas per default. This broke pocl on e.g. ARM for LLVM 3.9 (https://github.com/pocl/pocl/issues/409). Example: $ echo "#pragma OPENCL EXTENSION cl_khr_icd: enable" > hello.cl $ clang -emit-llvm -x cl -o tmp.bc -c hello.cl works fine, but: $ clang -emit-llvm -x cl -o tmp.bc -c hello.cl --target=armv7-unknown-linux-gnueabihf hello.cl:1:26: warning: unsupported OpenCL extension 'cl_khr_icd' - ignoring [-Wignored-pragmas] #pragma OPENCL EXTENSION cl_khr_icd: enable ^ 1 warning generated. Attached is a patch that enables OpenCL extensions for all targets per default, and then sets the status quo of supported extensions for those targets that currently customize their settings (i.e. NVPTX and AMDGPU). Most generic CPUs can handle all OpenCL extensions just fine. Please keep me as CC, I am not subscribed to the list. thanks, kalle ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ pocl-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/pocl-devel
