On 06/08/2014 10:11 PM, Erik Schnetter wrote:
> We currently provided some kernel functions for the half data type. For many 
> of
> these, LLVM would fail to generate machine code from byte code. I removed 
> those
> functions that the standard does not require.

Please do not remove the half functions!
I have just added some of those recently as we needed them
in our benchmark cases.

We use them heavily in our group where we design
cores that support native computation with halfs.

It's a standard extension in OpenCL:
http://www.khronos.org/registry/cl/sdk/1.0/docs/man/xhtml/cl_khr_fp16.html

The correct way is to disable them conditionally for devices that do
not support halfs natively (and thus fail to generate code for them).
Perhaps, in addition to __IF_FP16, we need __IF_NATIVE_FP16?

x86_64 doesn't support computation on halfs, but even in that case
it can be useful to emulate native computation to test kernels that
use halfs natively. But that's another story.

So, please revert until a proper fix is found (and before someone
in our group accidentally merges this and breaks our project cases).

-- 
--Pekka


------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their 
applications. Written by three acclaimed leaders in the field, 
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/NeoTech
_______________________________________________
pocl-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/pocl-devel

Reply via email to