On Tue, Jul 21, 2009 at 8:08 PM, Mon P Wang<[email protected]> wrote: > Author: wangmp > Date: Tue Jul 21 22:08:17 2009 > New Revision: 76717 > > URL: http://llvm.org/viewvc/llvm-project?rev=76717&view=rev > Log: > Preserve address space information through member accesses, e.g., > __attribute__((address_space(1))) struct {int arr[ 3 ]; } *p1; > ... = p1->arr[2]; // load from address space 1
You're missing tests for the BuildAnonymousStructUnionMemberReference; otherwise, the Sema changes look okay. But please try to add some tests that don't depend on CodeGen (a good way to verify types are correct is to declare an object by directly writing out the type, then re-declare it using an __typeof as the type). The CodeGen changes look wrong; the address-space should already be encoded in the type of the Value*, so it shouldn't be necessary to separately encode it explicitly. Also, as far as I can tell (correct me if I'm wrong), they don't actually do anything, since you didn't add any uses of the AddressSpace member. -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
