[cfe-dev] double/virtual offsets for MSVC broken in 3.1

[João Matos]

In my earlier mail, I actually posted the wrong dump without MS ABI.

Here is the correct one:

*** Dumping AST Record Layout
   0 | struct DoubleInPolymorphicStruct
   0 |   (DoubleInPolymorphicStruct vftable pointer)
   4 |   int a
   8 |   double b
  sizeof=16, dsize=16, align=8
  nvsize=16, nvalign=8

But the issue still exists.

I tried to apply the patch by Don but it's not compatible with trunk. You
seem to know a lot more about the layout code, so if you or Don could fix
it it would be the best.


On Fri, Sep 21, 2012 at 3:42 PM, r4start <r4start at gmail.com> wrote:

>  On 21/09/12 17:33, João Matos wrote:
>
> What is the status on this?
>
>  I just checked with trunk, and can verify the behavior Don is seeing.
>
>  *** Dumping AST Record Layout
>    0 | struct DoubleInPolymorphicStruct
>    0 |   (DoubleInPolymorphicStruct vtable pointer)
>    0 |   (DoubleInPolymorphicStruct vftable pointer)
>    4 |   int a
>    8 |   double b
>   sizeof=16, dsize=16, align=8
>    nvsize=16, nvalign=8
>
>  What test case did you use Dmitry?
>
>  Hi.
> Test case similar with Don example.
> I used -Xclang -cxx-abi -Xclang microsoft 1.cpp -Xclang
> -fdump-record-layouts,
> but the right way is -Xclang -cxx-abi -Xclang microsoft  -Xclang *
> -triple=i686-pc-win32* 1.cpp -Xclang -fdump-record-layouts.
> So the issue is still alive.
> Anybody work on it?
>
> -Dmitry Sokolov.
>



-- 
João Matos
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20120921/28b053f6/attachment.html>