[PATCH] OpenCL 1.2 allow explicit zero-to-event_t cast

Pekka Jääskeläinen pekka.jaaskelainen at tut.fi
Mon Sep 29 13:16:46 PDT 2014


Hi,

On 09/29/2014 07:06 PM, Sahasrabuddhe, Sameer wrote:
> But if such a cast is already present in a conformance test, I suppose every
> OpenCL vendor has a frontend (possibly Clang-based) that allows the explicit
> cast. Can this not be considered a "de facto" refinement to the standard? Or do
> we strictly need a clarification from Khronos?

OK. Seems to me the actual culprit is the conformance test that tests
undefined behavior.

However, my opinion is that the best way forward would be to commit the
patch to enable the current conformance test to pass with an upstream clang,
and point this to the Khronos group so they can either remove the test or
make it explicitly allowed in the specs. Hard to see why they would
explicitly forbid it.

-- 
--Pekka




More information about the cfe-commits mailing list