Incorrect equality check in Data_Interval_Duration_Iso

[mikesol]

The following cpp code was generated for boxed variables:

            return Math::floor()(a) != a;

I'm still learning how all this is put together, but it seems the correct generation would be:

            return Data_Eq::notEq()(Math::floor()(a))(a);

[andyarvanitis]

Like the other report, that's due to the inliner. Direct floating point comparisons are never really a good idea, though – I removed them from the golang backend, and should do the same for the C++ one. But was there a bug here, or is it just a question about the codegen?

[mikesol]

It's a bug, apologies for the ambiguity. It did not compile unless I changed it to return Data_Eq::notEq()(Math::floor()(a))(a);.

[andyarvanitis]

Ah ok, interesting. At a glance, I don't know why that isn't working. I'll likely go ahead and disable the inliner for Number (this is already there), but I'll have a look as to why first.