[PATCH] [ms-cxxabi] Implement member data pointers for non-dynamic classes
Reid Kleckner
rnk at google.com
Fri Mar 22 09:58:18 PDT 2013
On Thu, Mar 21, 2013 at 11:14 PM, John McCall <rjmccall at apple.com> wrote:
> All these tests are pretty inadequate; you could be accidentally doing
> a lot of stuff besides the stores. You should basically be testing the
> entire
> function body, including the define line. This will also let you
> effectively
> test that you're generating the right IR type, which will be important when
> you start generating different IR types for different cases.
>
OK, now the alloca and its size is part of the test. That should constrain
the size of the member pointer to a single slot.
> You're also not testing the generation of a null value.
>
How so? That's what the final store -1/0 is in each function.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130322/85b10335/attachment.html>
More information about the cfe-commits
mailing list