Updating Read to Parse Flonums and Fixing Flonum Compilation

[WChung43]

Hello. This is my small update to my implementation of the Floating point numbers for #1. I have updated read by doing a minor edit and uncommenting, so that the read file can parse Flonums, and I have fixed the compiler and a86 to remove bugs for boxing the result of float arithmetic operations during compilation.

The summary of the changes I made are:

read.rkt

• Uncommented and made a slight edit to the functions that read floats.

compile.rkt

• Previously there was a bug where a 128 bit was put in a 64 bit registers in fl+ and fl- which would cause errors. Fixed this by replacing all instances of Movapd with the correct instruction of Movsd

float.rkt

• Added this test file to test flonum operations now that the read function can parse flonums

Possible things to add next:

• Overflow errors

[8tx7K38ej1aBTKWK]

By the way, I discovered a bug in fl+ and fl- when testing the SDL integration.

The movapd instruction (>=128-bit copy) should be replaced by movsd (64-bit copy). A fix can be found here:

https://github.com/plum-umd/the-838e-compiler/commit/2dd764b535c650a39ae7fe069436c518f41471a5#diff-b6c8fa84f32f5d7e9930a605d234098840001abc94ed39344ae0bc0b4a5f2927R734-R752

[WChung43]

By the way, I discovered a bug in fl+ and fl- when testing the SDL integration.

The movapd instruction (>=128-bit copy) should be replaced by movsd (64-bit copy). A fix can be found here:

2dd764b#diff-b6c8fa84f32f5d7e9930a605d234098840001abc94ed39344ae0bc0b4a5f2927R734-R752

Don't worry. I saw that bug when testing the read function and I fixed it as part of this pull request in the edits for compile.rkt.