Skip to content

Conversation

@jayrm
Copy link
Member

@jayrm jayrm commented Oct 25, 2020

These changes were required to have the test-suite pass on Linux & Windows 64-bit

  • add hIsStructIn2Regs() internal function
  • add comments in hGetReturnTypeGas64Linux()
  • invert test for SSE 41 feature flag - my CPU doesn't have SSE41 and executables were failing with invalid instructions

Had some troubles with PR #260, causing new errors on windows. This PR should gets things back to normal for @SARG-FB to continue development.

jayrm added 4 commits October 24, 2020 23:44
This reverts commit 4d177a8.

Revert "gas64: fix check"

This reverts commit 350fd28.
- add hIsStructIn2Regs() internal function
- add comments in hGetReturnTypeGas64Linux()
@jayrm jayrm merged commit 91ff104 into freebasic:master Oct 25, 2020
jayrm added a commit to jayrm/fbc that referenced this pull request Oct 25, 2020
(cherry picked from commit 5267093)

gas64: fix check
(cherry picked from commit 350fd28)

gas64: fix internal check

- returning structure values in registers true only if type is non-pointer struct
(cherry picked from commit 4d177a8)

Merge pull request freebasic#260 from jayrm/fbc-gas64

change gas64 internal checks for structs returned in registers on linux
(cherry picked from commit fbedc0a)

Revert "travis: disable some tests for faster build testing"

This reverts commit 5267093.
(cherry picked from commit 24d25c6)

Revert "gas64: fix internal check"

This reverts commit 4d177a8.

Revert "gas64: fix check"

This reverts commit 350fd28.
(cherry picked from commit 83217e8)

gas64: check if structure is returned in 2 registers

- add hIsStructIn2Regs() internal function
- add comments in hGetReturnTypeGas64Linux()
(cherry picked from commit e973a9f)

gas64: invert test for SSE 41 feature flag
(cherry picked from commit c15753f)

Merge pull request freebasic#261 from jayrm/fbc-gas64

gas64: check if structure is returned in 2 registers for linux
(cherry picked from commit 91ff104)
@jayrm jayrm mentioned this pull request Oct 25, 2020
@jayrm jayrm deleted the fbc-gas64 branch October 25, 2020 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant