Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDL_cpuinfo.c does not compile on Solaris / GCC / Sun assembler #454

Closed
SDLBugzilla opened this issue Feb 10, 2021 · 0 comments
Closed

SDL_cpuinfo.c does not compile on Solaris / GCC / Sun assembler #454

SDLBugzilla opened this issue Feb 10, 2021 · 0 comments
Labels
wontfix This will not be worked on

Comments

@SDLBugzilla
Copy link
Collaborator

SDLBugzilla commented Feb 10, 2021

This bug report was migrated from our old Bugzilla tracker.

Reported in version: 1.2.13
Reported for operating system, platform: Solaris, Other

Comments on the original bug report:

On 2009-01-22 08:36:18 +0000, Jörn Clausen wrote:

Compiling SDL_cpuinfo.c on Solaris 10/x86 with GCC 3.4.6 and /usr/ccs/bin/as stops with

 gcc -g -O2 -I/usr/local/pkgsrc/pkgsrc-cvs/include -I./include -D_GNU_SOURCE=1 -I/usr/local/pkgsrc/pkgsrc-cvs/include -I/usr/local/pkgsrc/pkgsrc-cvs/include -DXTHREADS -D_REENTRANT -c ./src/cpuinfo/SDL_cpuinfo.c  -fPIC -DPIC -o build/.libs/SDL_cpuinfo.o
Assembler: SDL_cpuinfo.c
        "/var/tmp//ccRMHeed.s", line 57 : Syntax error
        Near line: "            pushfl                      # Get original EFLAGS             "
        "/var/tmp//ccRMHeed.s", line 60 : Syntax error
        Near line: "        xorl    $0x200000,%eax     # Flip ID bit in EFLAGS           "
        "/var/tmp//ccRMHeed.s", line 61 : Syntax error
        Near line: "        pushl   %eax               # Save new EFLAGS value on stack  "
        "/var/tmp//ccRMHeed.s", line 62 : Syntax error
        Near line: "        popfl                       # Replace current EFLAGS value    "
        "/var/tmp//ccRMHeed.s", line 63 : Syntax error
        Near line: "        pushfl                      # Get new EFLAGS                  "
        "/var/tmp//ccRMHeed.s", line 64 : Syntax error
        Near line: "        popl    %eax               # Store new EFLAGS in EAX         "
        "/var/tmp//ccRMHeed.s", line 65 : Syntax error
        Near line: "        xorl    %ecx,%eax         # Can not toggle ID bit,          "
        "/var/tmp//ccRMHeed.s", line 66 : Syntax error
        Near line: "        jz      1f                  # Processor=80486                 "
        "/var/tmp//ccRMHeed.s", line 67 : Syntax error

Trying to configure SDL with --disable-cpuinfo didn't help. As far as I can see, the define SDL_CPUINFO_DISABLED is set, but never checked. --disable-assembly doesn't help either, as the define SDL_ASSEMBLY_ROUTINES is not checked in SDL_cpuinfo.c. Trying to activate other assembler code (the one defined after #elif defined(__sun) && defined(__i386)) resulted in a successful compile, but the resulting library segfaults in this function.

On 2009-09-13 16:33:23 +0000, Ryan C. Gordon wrote:

Tagging this bug with "target-1.2.14" so we can try to resolve it for SDL 1.2.14.

Please note that we may choose to resolve it as WONTFIX. This tag is largely so we have a comprehensive wishlist of bugs to examine for 1.2.14 (and so we can close bugs that we'll never fix, rather than have them live forever in Bugzilla).

--ryan.

On 2009-09-21 03:10:06 +0000, Sam Lantinga wrote:

We're not planning to fix this ourselves, though patches are welcome!

@SDLBugzilla SDLBugzilla added bug wontfix This will not be worked on labels Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant