Re: PATCH: re-enable OpenCL extensions

2017-01-20 Thread Kalle Raiskila via cfe-commits
On 19.01.2017 23:43, Liu, Yaxun (Sam) wrote: 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 agree on the first part, but arrive at the exact opposite result

RE: PATCH: re-enable OpenCL extensions

2017-01-19 Thread Liu, Yaxun (Sam) via cfe-commits
different targets based on that. Sam -Original Message- From: Anastasia Stulova [mailto:anastasia.stul...@arm.com] Sent: Thursday, January 19, 2017 12:31 PM To: Kalle Raiskila Cc: cfe-commits@lists.llvm.org; Liu, Yaxun (Sam) ; nd Subject: RE: PATCH: re-enable OpenCL extensions As me

RE: PATCH: re-enable OpenCL extensions

2017-01-19 Thread Anastasia Stulova via cfe-commits
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