[PATCH] Add address space arguments to MemoryBuiltins
Nuno Lopes
nunoplopes at sapo.pt
Thu Oct 3 11:33:11 PDT 2013
The patch itself LGTM.
The test, however, doesn't look right. Doesn't it pass without the patch? You need a test case that really shows that the patch is working. E.g., two objectsize on pointers with different sizes, objectize returning an i32, when the pointer is only 8 bits (and vice-versa), etc. All those tricky edge cases.
http://llvm-reviews.chandlerc.com/D1685
More information about the llvm-commits
mailing list