cancel
Showing results for 
Search instead for 
Did you mean: 

Archives Discussions

zeland
Journeyman III

cl_float3

typedef cl_float4 cl_float3; (c) cl_platform.h

I've found in win XP sp3 x86 32 bit ATI stream SDK v2.3  cl_platform.h

follow code:

/* cl_float3 is identical in size, alignment and behavior to cl_float4. See section 6.1.5. */
typedef cl_float4 cl_float3;

IHMO this is wrong way...

 

0 Likes
5 Replies
MicahVillmow
Staff
Staff

cl_float3

How so? cl_float3 is only created as a convenience type. Except for a lack of a w channel in a kernel, it is exactly the same as cl_float4.
0 Likes
zeland
Journeyman III

cl_float3

But  addressing to .w in cl_float3 must be a error.

0 Likes
LeeHowes
Staff
Staff

cl_float3

Addressing to w needn't give an error, really. What is important is that if you pass float3 to function calls they behave correctly. For example, a dot product on a float3 is not the same as a dot product on a float4. If w is initialised to 0 this isn't really a problem barring marginally inefficient code generation. As long as once the data is on the device it is treated as a true float3, having the host cost correctly deal with the alignment characteristics is the most important thing, surely?

How else would you deal with it? Create a host structure containing x, y, z, _padding? Or just have x, y, z and hope compiler alignment guarantees are effective (which is something I don't have much faith in).

0 Likes
MicahVillmow
Staff
Staff

cl_float3

In a kernel it must be an error, not in the host program, as the spec only defines the kernel language definition. From the spec:
"float3 pos;
pos.z = 1.0f; // is legal
pos.w = 1.0f; // is illegal"
This only is illegal for a float3, not a cl_float3.
On a side note, the header file that defines cl_float3 is straight from Khronos, not something that we create. So if it is behavior you want changed, you need to go through Khronos to get it changed.
See here:
http://www.khronos.org/registr.../api/1.1/cl_platform.h
0 Likes
zeland
Journeyman III

cl_float3

Thanks I get it. Really on http://www.khronos.org/registry/cl/api/1.1/cl_platform.h code is the same.

0 Likes