This change removes the "XOR NaN" hack. The hack was initially introduced as a performance optimisation, but is fairly complicated.
NOTE: this PR introduces a performance regression of up to 5% for some operations. The performance regression will be fixed in a subsequent PR my modifying how the Envelope method for LineStrings is implemented.
peterstace
commented
1 year ago
Description
This change removes the "XOR NaN" hack. The hack was initially introduced as a performance optimisation, but is fairly complicated.
NOTE: this PR introduces a performance regression of up to 5% for some operations. The performance regression will be fixed in a subsequent PR my modifying how the
Envelopemethod forLineStrings is implemented.Check List
Have you:
Added unit tests? Relies on existing.
Add cmprefimpl tests? (if appropriate?) N/A.
Updated release notes? (if appropriate?) Yes.
Related Issue
Benchmark Results
results.txt