Cannot build ghc HEAD with LLVM ARM due to globalRegMaybe

[Moritz Angermann]

Dear Simon,

No, there is not. Should every patch in Phab, have an associated ticket as well?

Cheers,
 Moritz

On Sep 15, 2014, at 2:17 PM, Simon Peyton Jones <simonpj at microsoft.com> wrote:

> Is there a Trac ticket too?
> 
> Simon
> 
> | -----Original Message-----
> | From: ghc-devs [mailto:ghc-devs-bounces at haskell.org] On Behalf Of Moritz
> | Angermann
> | Sent: 15 September 2014 10:19
> | To: ghc-devs at haskell.org
> | Subject: Re: Cannot build ghc HEAD with LLVM ARM due to globalRegMaybe
> | 
> | Hi Janek,
> | 
> | yea :) I think it went by unnoticed.  Patch is already in Phab,
> | 
> | https://phabricator.haskell.org/D208
> | 
> | Not sure how to speed up the review process, though ;-)
> | 
> | Cheers,
> |   Moritz
> | 
> | On Sep 15, 2014, at 11:13 AM, Jan Stolarek <jan.stolarek at p.lodz.pl>
> | wrote:
> | 
> | > I think it is best to fill in a bug report and attach a patch - I'm
> | afraid this mail may have gone
> | > unnoticed :-)
> | >
> | > Janek
> | >
> | > Dnia piątek, 12 września 2014, Moritz Angermann napisał:
> | >> Hi,
> | >>
> | >> when trying to compile a ghc HEAD on LLVM ARM, I end up getting an
> | >> exception:
> | >>
> | >>    ghc-stage1: panic! (the 'impossible' happened)
> | >>      (GHC version 7.9.20140911 for arm-apple-ios):
> | >>        globalRegMaybe
> | >>
> | >> After some digging I found that a new call to globalRegMaybe was
> | introduced
> | >> here:
> | >> http://git.haskell.org/ghc.git/blob/HEAD:/compiler/cmm/CmmSink.hs#l231
> | on
> | >> 2014-04-29 in commit:
> | >>
> | http://git.haskell.org/ghc.git/commitdiff/83a003fcaec93dbfd5b46837f2bf335
> | 34
> | >> 12b9877
> | >>
> | >> In a similar timeframe (since 2014-05-02) /include/CodeGen.Platform.hs
> | was
> | >> changed:
> | >>
> | http://git.haskell.org/ghc.git/blobdiff/b0534f78a73f972e279eed4447a5687bd
> | 6a
> | >> 8308e..HEAD:/includes/CodeGen.Platform.hs
> | >>
> | >> introducing a set of new ifdef/elif/endif pairs. Especially for
> | >> MACHREGS_NO_REGS, namely:
> | >>
> | >>    -#endif
> | >>     globalRegMaybe _                        = Nothing
> | >>    +#elif MACHREGS_NO_REGS
> | >>    +globalRegMaybe _ = Nothing
> | >>    +#else
> | >>    +globalRegMaybe = panic "globalRegMaybe not defined for this
> | platform"
> | >>    +#endif
> | >>
> | >> ARM.hs
> | >>
> | (http://git.haskell.org/ghc.git/blob/23892440032fcab8a6a753916d5068576741
> | 69
> | >> ec:/compiler/codeGen/CodeGen/Platform/ARM.hs) however, has:
> | >>
> | >>    #define MACHREGS_NO_REGS 0
> | >>    #define MACHREGS_arm 1
> | >>
> | >> where MACHREGS_arm 1 has no effect on the subsequently imported
> | >> CodeGen.Platform.hs.
> | >>
> | >> Therefore I propose that a change to ARM.hs is made to set
> | >> MACHREGS_NO_REGS to 1.
> | >>
> | >> Cheers,
> | >> Moritz
> | >> _______________________________________________
> | >> ghc-devs mailing list
> | >> ghc-devs at haskell.org
> | >> http://www.haskell.org/mailman/listinfo/ghc-devs
> | >
> | >
> | 
> | _______________________________________________
> | ghc-devs mailing list
> | ghc-devs at haskell.org
> | http://www.haskell.org/mailman/listinfo/ghc-devs