[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lmi] rounding unit test under Linux/x86-64

From: Vadim Zeitlin
Subject: Re: [lmi] rounding unit test under Linux/x86-64
Date: Thu, 16 Jun 2011 18:41:17 +0200

On Thu, 16 Jun 2011 14:23:12 +0000 Greg Chicares <address@hidden> wrote:

GC> On 2010-09-28 00:33Z, Vadim Zeitlin wrote:
GC> > 
GC> >  I finally have a sequence of patches which implement the proposed 
GC> > For now I've uploaded them at http://review.bakefile.org/r/231/
GC> With this change, 'fenv_lmi_x86.hpp' loses the property tested by
GC> 'make check_physical_closure' (and described in section 3.4 of
GC> Lakos, _Large-Scale C++ Software Design_), and now this test:
GC>   g++ -x c++ -w -O0 -fsyntax-only fenv_lmi_x86.hpp
GC> produces these:
GC>   /lmi/src/lmi/fenv_lmi_x86.hpp:135: error: `e_ieee754_precision' does not 
name a type
GC>   /lmi/src/lmi/fenv_lmi_x86.hpp:136: error: `e_ieee754_rounding' does not 
name a type
GC> and other consequent diagnostics.

 Sorry, I wasn't aware of this test and so didn't run it. Do I understand
correctly that I would have needed to run "make test_various_build_types"
to detect this problem?

GC> Is there a good alternative to moving those enumerations from
GC> 'fenv_lmi_x86.hpp' to 'fenv_lmi.hpp'?

 The simplest would probably be to simply include fenv_lmi.hpp from
fenv_lmi_x86.hpp, wouldn't it? While this is contrary to how it used be
done, I don't see any serious problem with doing it.

 Am I missing something?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]