[Beignet] [PATCH] GBE: switch to use CLANG native image types.

Zhigang Gong zhigang.gong at linux.intel.com
Wed Dec 17 23:00:38 PST 2014


On Thu, Dec 18, 2014 at 07:41:55AM +0000, Yang, Rong R wrote:
> One minor comment.
> 

> > +        if (llvmInfo.isImageType()) {
> > +          ctx.input(argName, ir::FunctionArgument::IMAGE, reg, llvmInfo, 4, 4,
> > 0);
> [Yang, Rong R] Is it safe to hard code the size to 4?

It is safe if we haven't switch to 64 bit pointer. But it's not good to hard
coded it to 4. I will change it to use pointersize rather than 4.

Thanks for the comment.

> 
> 


More information about the Beignet mailing list