Hi Jingyue, I committed the addrspacecast isel patterns to NVPTX. Also, I wanted to point out that your changes in the last test case in this patch ( address-spaces.cu) represent changes that may lead to performance degradation. Specific address spaces should be used whenever possible for loads/stores. Casting everything to a generic address is still correct, but may lead to additional indirections for the hardware.
On Fri, Mar 21, 2014 at 2:25 PM, Justin Holewinski <[email protected]>wrote: > addrspacecast support in NVPTX is on my todo list. I'll try to put > something together in the next few days. > > > On 3/21/14, 2:20 PM, Jingyue Wu wrote: > > Hi, > > Static local variables in CUDA can be declared with address space > qualifiers, such as __shared__. Therefore, the codegen needs to potentially > addrspacecast a static local variable to the type expected by its > declaration. Peter did something similar for global variables in r157167. > > All clang tests passed. > > Justin: The NVPTX backend support for addrspacecast seems not complete. > We can send you follow-up patches once this one gets in. > > Jingyue > > > > -- > Thanks, > > Justin Holewinski > > ------------------------------ > This email message is for the sole use of the intended recipient(s) and > may contain confidential information. Any unauthorized review, use, > disclosure or distribution is prohibited. If you are not the intended > recipient, please contact the sender by reply email and destroy all copies > of the original message. > ------------------------------ > -- Thanks, Justin Holewinski
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
