[m-rev.] for review: fix float field structure padding/packing problems
Peter Wang
novalazy at gmail.com
Mon Dec 5 20:11:21 AEDT 2011
On Mon, 5 Dec 2011 18:57:47 +1100 (EST), Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
>
> Hi Peter,
>
> On Mon, 5 Dec 2011, Peter Wang wrote:
>
> > Can someone apply this and test with MSVC and MinGW?
>
> Shall do. Has it been tested with clang?
Not yet.
> > I've only tested the single precision floats on 64-bit Linux part.
> >
> > Would it be worth avoiding the C extensions when
> > sizeof(MR_Float) == sizeof(MR_Word)? That can't be evaluated by the C
> > preprocessor, though.
>
> The configure script could make that test and define
> MR_FLOAT_IS_WORD_SIZED or something if that condition is true.
No, it depends on the grade being targeted.
> I'll review the rest later - although could you please go ahead and commit the
> test case for bug #240 now.
Done.
Peter
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to: mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions: mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------
More information about the reviews
mailing list