#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: new Priority: normal | Milestone: Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Description changed by nomeata: Old description:

Hello!

I have successfully bootstrapped ghc for sparc64 on Debian unstable by cross-compiling it on amd64, then installing it manually into a build root and rebuild the Debian package.

Unfortunately, the buildds can currently neither compile ghc itself nor any Haskell packages as gcc always passes "-relax" to the GNU linker which will fail as ghc passes "-Wl,-r" [1]:

"/usr/bin/ghc" -hisuf hi -osuf o -hcsuf hc -static -H32m -O -lffi -optl-pthread -package-db libraries/bootstrapping.conf -this-package- key termi_6iVf4EBnOgfIaaOCLRs8jl -hide-all-packages -i -ilibraries/terminfo/. -ilibraries/terminfo/dist-boot/build -ilibraries/terminfo/dist-boot/build/autogen -Ilibraries/terminfo/dist- boot/build -Ilibraries/terminfo/dist-boot/build/autogen -Ilibraries/terminfo/. -optP-include -optPlibraries/terminfo/dist- boot/build/autogen/cabal_macros.h -package-key base_HQfYBxpPvuw8OunzQu6JGM -Wall -XHaskell2010 -no-user-package-db -rtsopts -odir libraries/terminfo/dist-boot/build -hidir libraries/terminfo/dist-boot/build -stubdir libraries/terminfo/dist- boot/build -c libraries/terminfo/./System/Console/Terminfo/Base.hs -o libraries/terminfo/dist-boot/build/System/Console/Terminfo/Base.o /usr/bin/ld: --relax and -r may not be used together collect2: error: ld returned 1 exit status make[3]: *** [libraries/terminfo/dist- boot/build/System/Console/Terminfo/Base.o] Error 1 libraries/terminfo/ghc.mk:3: recipe for target 'libraries/terminfo/dist- boot/build/System/Console/Terminfo/Base.o' failed

This issue has already been fixed in ghc but that was for sparc only and not sparc64 [2].

Thus, in order to be able to apply the same fix from [2], ghc needs to add sparc64 to its list of known architectures (currently "sparc64" matches to "ArchUnknown" in aclocal.m4") and then extend the fix from [2] to match *both* sparc *and* sparc64 (please don't drop sparc here).

In particular, I suggest adding sparc64 in the same way that Alpha, Mipseb and Mipsel were added [3], thus matching "sparc64" to "ArchSPARC64", then extending the if-clause from [4] to set "-Wl,-no- relax" for *both* ArchSPARC *and* ArchSPARC64.

Thanks, Adrian

...

[1] https://buildd.debian.org/status/fetch.php?pkg=ghc&arch=sparc64&ver=7.10.3-3&stamp=1449983921 [2] https://ghc.haskell.org/trac/ghc/ticket/3791 [3] https://git.haskell.org/ghc.git/commitdiff/9756690fe7aa26aee6955d0b720377d53... [4] https://git.haskell.org/ghc.git/commitdiff/3b322660f82d0c7c4f7d02523367ebd0e...

New description: Hello! I have successfully bootstrapped ghc for sparc64 on Debian unstable by cross-compiling it on amd64, then installing it manually into a build root and rebuild the Debian package. Unfortunately, the buildds can currently neither compile ghc itself nor any Haskell packages as gcc always passes "-relax" to the GNU linker which will fail as ghc passes "-Wl,-r" (1): {{{ "/usr/bin/ghc" -hisuf hi -osuf o -hcsuf hc -static -H32m -O -lffi -optl- pthread -package-db libraries/bootstrapping.conf -this-package-key termi_6iVf4EBnOgfIaaOCLRs8jl -hide-all-packages -i -ilibraries/terminfo/. -ilibraries/terminfo/dist-boot/build -ilibraries/terminfo/dist- boot/build/autogen -Ilibraries/terminfo/dist-boot/build -Ilibraries/terminfo/dist-boot/build/autogen -Ilibraries/terminfo/. -optP-include -optPlibraries/terminfo/dist- boot/build/autogen/cabal_macros.h -package-key base_HQfYBxpPvuw8OunzQu6JGM -Wall -XHaskell2010 -no-user-package-db -rtsopts -odir libraries/terminfo/dist-boot/build -hidir libraries/terminfo/dist- boot/build -stubdir libraries/terminfo/dist-boot/build -c libraries/terminfo/./System/Console/Terminfo/Base.hs -o libraries/terminfo /dist-boot/build/System/Console/Terminfo/Base.o /usr/bin/ld: --relax and -r may not be used together collect2: error: ld returned 1 exit status make[3]: *** [libraries/terminfo/dist- boot/build/System/Console/Terminfo/Base.o] Error 1 libraries/terminfo/ghc.mk:3: recipe for target 'libraries/terminfo/dist- boot/build/System/Console/Terminfo/Base.o' failed }}} This issue has already been fixed in ghc but that was for sparc only and not sparc64 (2). Thus, in order to be able to apply the same fix from (2), ghc needs to add sparc64 to its list of known architectures (currently "sparc64" matches to "ArchUnknown" in aclocal.m4") and then extend the fix from (2) to match *both* sparc *and* sparc64 (please don't drop sparc here). In particular, I suggest adding sparc64 in the same way that Alpha, Mipseb and Mipsel were added (3), thus matching "sparc64" to "ArchSPARC64", then extending the if-clause from (4) to set "-Wl,-no-relax" for *both* ArchSPARC *and* ArchSPARC64. Thanks, Adrian * (1) https://buildd.debian.org/status/fetch.php?pkg=ghc&arch=sparc64&ver=7.10.3-3&stamp=1449983921 * (2) https://ghc.haskell.org/trac/ghc/ticket/3791 * (3) https://git.haskell.org/ghc.git/commitdiff/9756690fe7aa26aee6955d0b720377d53... * (4) https://git.haskell.org/ghc.git/commitdiff/3b322660f82d0c7c4f7d02523367ebd0e... -- -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211#comment:2 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: new Priority: normal | Milestone: Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Changes (by glaubitz): * Attachment "0002-Explicitly-pass-no-relax-on-ArchSPARC64-where- gcc-s-.patch" added. Explicitly pass "-no-relax" to gcc on ArchSPARC64. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: new Priority: normal | Milestone: Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Changes (by glaubitz): * Attachment "0001-Map-sparc64-to-ArchSPARC64-instead-of-ArchUnknown- an.patch" added. Replacement for 0001-Map-sparc64-to-ArchSPARC64-instead-of- ArchUnknown.patch -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: new Priority: normal | Milestone: Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Changes (by glaubitz): * Attachment "0002-Explicitly-pass-no-relax-on-ArchSPARC64-where- gcc-s-.patch" added. Reformatted the second patch to adapt ghc coding style better. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: patch Priority: highest | Milestone: 8.0.1 Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Comment (by glaubitz): Hi thomie! I just realized I forgot patching additional files in the compiler, namely compiler/cmm/PprC.hs compiler/nativeGen/AsmCodeGen.hs compiler/nativeGen/RegAlloc/Graph/TrivColorable.hs compiler/nativeGen/RegAlloc/Linear/FreeRegs.hs compiler/nativeGen/RegAlloc/Linear/Main.hs compiler/nativeGen/TargetReg.hs where the architecture is checked in case statements as well. I have already patched my tree and doing some test builds now, both on amd64 and sparc64. The latter will take some more time though :(. Adrian -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211#comment:6 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+---------------------------------- Reporter: glaubitz | Owner: Type: bug | Status: infoneeded Priority: highest | Milestone: 8.0.1 Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+---------------------------------- Changes (by glaubitz): * Attachment "0001-Add-the-initial-pieces-to-make-sparc64-a-known- archi.patch" added. Initial pieces for sparc64 platform support. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+---------------------------------- Reporter: glaubitz | Owner: Type: bug | Status: infoneeded Priority: highest | Milestone: 8.0.1 Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+---------------------------------- Comment (by glaubitz): Alright, I have now successfully tested my modified patches, full build log for ghc 7.10.3 with the patches applied on sparc64 here:

https://buildd.debian.org/status/fetch.php?pkg=ghc&arch=sparc64&ver=7.10.3-4&stamp=1450393450 Please review my two *latest* patches 0001-Add-the-initial-pieces-to-make-sparc64-a-known-archi.patch and 0002-On-sparc64-gcc-has-mrelax-in-it-s-default-specs-so-w.patch and apply them once you're happy. Thanks, Adrian -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211#comment:8 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: patch Priority: highest | Milestone: 8.0.1 Component: Compiler | Version: 7.10.2 Resolution: | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Comment (by glaubitz): So, I did a git-grep and I don't think the other occurences are relevant for sparc64. For one, compiler/main/DriverPipeline.hs has several tests for ArchSPARC and then passes -mcpu=v9 to gcc which is irrelevant for sparc64 which always defaults -mcpu to "v9" unlike sparc which, by default, matches to -mcpu=v8. The other occurences in question are in libraries/ghci/GHCi/InfoTable.hsc. However, since there is no NGC on sparc64 (yet), I don't think ghci is supported at all, is it? My understanding is that ghci always requires NGC support but I might be wrong. The mention in compiler/llvmGen/LlvmCodeGen/CodeGen.hs seems to apply for NGC-suppored architectures only as well, is it? Basically, my patches just add the necessary changes to make sparc64 known to ghc analog to https://git.haskell.org/ghc.git/commitdiff/9756690fe7aa26aee6955d0b720377d53..., plus the SPARC-related change https://git.haskell.org/ghc.git/commitdiff/3b322660f82d0c7c4f7d02523367ebd0e... which enforces -no-relax. NGC will add some additional mentions of ArchSPARC64 in code in the future. And since GHC's current NGC for SPARC already implements the SPARC V9 ISA but just with 32-bit pointers, it shouldn't be too difficult to add NGC to sparc64 in the future. The current SPARC NGC probably just needs to extended or cloned plus modified for 64-bit pointers. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211#comment:10 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#11211: Please add initial platform support for sparc64 ----------------------------------------+------------------------------ Reporter: glaubitz | Owner: Type: bug | Status: closed Priority: highest | Milestone: 8.0.1 Component: Compiler | Version: 7.10.2 Resolution: fixed | Keywords: Operating System: Linux | Architecture: Other Type of failure: Building GHC failed | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | ----------------------------------------+------------------------------ Changes (by slyfox): * status: patch => closed * resolution: => fixed Comment: Pushed almost as-is: - added missing case clause detected by `./validate` build in `compiler/nativeGen/RegAlloc/Linear/FreeRegs.hs` - used `elem [ArchSPARC, ArchSPARC64]` to check for `-no-relax` workaround Thank you! -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/11211#comment:12 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler