Hi Tobias,

>> here is a fix for class pointer initialization.
>
> I think the patch looks reasonable.

well, it may appear so ...


> Additionally, the CLASS are wrongly initialized: You only set "_data"
> (indirectly as it is the first field/component of the class) but you do not
> set the _vptr component.

... but as you point out, it clearly produces wrong code :(

The attached new version should do the right thing now. At least it
shows the correct dump for the original test case as well as yours. It
is currently being regtested.


> PS: The test case. With module - fails at run time for same_type_as. Without
> module - one gets an ICE.

This now gives the correct run-time behavior with the module, but
still ICEs without. Unfortunately I have no idea why ...

Anyway, thanks for your review.

Cheers,
Janus

Attachment: pr57306_v3.diff
Description: Binary data

Reply via email to