Tables for common benchmark files:

Hardware: Intel Core i7-4771 (Haswell), Gigabyte GA-Z87P-D3, Kingston 9905403-831.A00LF 8GB x 2

Software: Ubuntu 14.10, Linux Kernel 3.16, GCC 4.9

I tested this diff and it renders at same speed in CUDA than before, and in CPU also same speed in my old amd quad core.

So because I don't see any loss in speed and because the new code is smaller I would commit it.

But instead to have these ".x" ".y" ".z" ".w" in the code, I would change the

struct ccl_try_align(16) int4 {

in util_types.h so it has this union instead:

union {

		__m128i m128;
		struct { int x, y, z, w; };
		struct { int type, visibility, index, object; };

};

and then to change all those ".x" ".y" ".z" ".w" using the names ".type" ".visibility", etc that makes more readable the code.

Is this patch still relevant after all those optimizations and changes being done?