[m-rev.] diff: update float_consistency test
Simon Taylor
stayl at cs.mu.OZ.AU
Thu Jun 12 15:06:21 AEST 2003
On 12-Jun-2003, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
>
> Estimated hours taken: 0.1
> Branches: main, release
>
> tests/hard_coded/float_consistency.m:
> Change calls to float./ to float.unchecked_quotient.
> This test case is meant to test compile-time evaluation
> of builtins. float./ hasn't been builtin for a while.
> This test case fails with --optimize-constant-propagation.
This wasn't right -- compiler/const_prop.m treats float./ as
if it were a builtin.
The cause of the failure appears to be different rounding
behaviour on earth (with gcc 3.2) depending on whether the
division is done inline or by a separate procedure.
Passing `-ffloat-store' to gcc (which disables storing
intermediate floats in registers) causes the test to pass.
Backing out my change to float_consistency.m and adding
`MCFLAGS-float_consistency = --no-intermodule-optimization'
to Mercury.options will fix the test failure, but would it
be better to make `-ffloat-store' the default?
Simon.
--------------------------------------------------------------------------
mercury-reviews mailing list
post: mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------
More information about the reviews
mailing list