Seems ok.

-Tanya

On Mar 7, 2013, at 3:38 AM, "Benyei, Guy" <[email protected]> wrote:

> Hi all,
> Any remarks on this, or can I go on and commit it? It’s quite a small change, 
> and it fixes a painful crash in OpenCL.
>  
> Thanks
>     Guy
> <image001.png>
>  
> From: [email protected] 
> [mailto:[email protected]] On Behalf Of Benyei, Guy
> Sent: Thursday, February 28, 2013 15:56
> To: [email protected]
> Subject: [PATCH] Address spaced pointers to structs
>  
> Hi all,
> This patch fixes a case, where a memory buffer of structs is located in an 
> address space different from 0, and one of its elements is being passed to a 
> function that expects a non-address spaced struct argument. In this case, the 
> value being passed should be copied to local memory, rather than trying to 
> pass its address in the address spaced memory object, which leads to a 
> failure.
>  
> Please review
>  
> Thanks
>    Guy
> <image001.png>
>  
> ---------------------------------------------------------------------
> Intel Israel (74) Limited
> 
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.
> 
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to