1. 14 Oct, 2021 6 commits
    • Matthew Pickering's avatar
      Some extra strictness in annotation fields · 481e6b54
      Matthew Pickering authored and  Marge Bot's avatar Marge Bot committed
      Locations can be quite long-lived so it's important that things which
      live in locations, such as annotations are forced promptly. Otherwise
      they end up retaining the entire PState, as evidenced by this retainer
      PState 0x4277ce6cd8 0x4277ce6d00 0x7f61f12d37d8 0x7f61f12d37d8 0x7f61f135ef78 0x4277ce6d48 0x4277ce6d58 0x4277ce6d70 0x4277ce6d58 0x4277ce6d88 0x4277ce6da0 0x7f61f29782f0 0x7f61cd16b440 0x7f61cd16b440 0x7f61d00f8d18 0x7f61f296d290 0x7f61cd16b440 0x7f61d00f8d18 0x7f61cd16b4a8 0x7f61f135ef78 0x4277ce6db8 0x4277ce6dd0 0x7f61f134f358 0 3 <PState:GHC.Parser.Lexer:_build-ipe/stage1/compiler/build/GHC/Parser/Lexer.hs:3779:46>
      _thunk(  ) 0x4277ce6280 0x4277ce68a0 <([LEpaComment], [LEpaComment]):GHC.Parser.Lexer:>
      _thunk(  ) 0x4277ce6568 <EpAnnComments:GHC.Parser.Lexer:compiler/GHC/Parser/Lexer.x:2306:19-40>
      _thunk(  ) 0x4277ce62b0 0x4277ce62c0 0x4277ce6280 0x7f61f287fc58 <EpAnn AnnList:GHC.Parser:_build-ipe/stage1/compiler/build/GHC/Parser.hs:12664:13-32>
      SrcSpanAnn 0x4277ce6060 0x4277ce6048 <SrcSpanAnn':GHC.Parser:_build-ipe/stage1/compiler/build/GHC/Parser.hs:12664:3-35>
      L 0x4277ce4e70 0x428f8c9158 <GenLocated:GHC.Data.BooleanFormula:compiler/GHC/Data/BooleanFormula.hs:40:23-29>
      0x428f8c8318 : 0x428f8c8300 <[]:GHC.Base:libraries/base/GHC/Base.hs:1316:16-29>
      Or 0x428f8c7890 <BooleanFormula:GHC.Data.BooleanFormula:compiler/GHC/Data/BooleanFormula.hs:40:23-29>
      IfConcreteClass 0x7f61cd16b440 0x7f61cd16b440 0x428f8c7018 0x428f8c7030 <IfaceClassBody:GHC.Iface.Make:compiler/GHC/Iface/Make.hs:(640,12)-(645,13)>
      Making these few places strict is sufficient for now but there are
      perhaps more places which will need strictifying in future.
      Metric Increase:
    • Alan Zimmerman's avatar
      EPA: Preserve semicolon order in annotations · 8b7f5424
      Alan Zimmerman authored and  Marge Bot's avatar Marge Bot committed
      Ensure the AddSemiAnn items appear in increasing order, so that if
      they are converted to delta format they are still in the correct
      Prior to this the exact printer sorted by Span, which is meaningless
      for EpaDelta locations.
    • Joachim Breitner's avatar
      fuzzyLookup: More deterministic order · f450e948
      Joachim Breitner authored and  Marge Bot's avatar Marge Bot committed
      else the output may depend on the input order, which seems it may depend
      on the concrete Uniques, which is causing headaches when including test
      cases about that.
    • Ziyang Liu's avatar
      Suggest -dynamic-too in failNonStd when applicable · 557d26fa
      Ziyang Liu authored and  Marge Bot's avatar Marge Bot committed
      I encountered an error that says
      Cannot load -dynamic objects when GHC is built the normal way
      To fix this, either:
        (1) Use -fexternal-interpreter, or
        (2) Build the program twice: once the normal way, and then
            with -dynamic using -osuf to set a different object file suffix.
      Or it could say
      (2) Use -dynamic-too
    • Joachim Breitner's avatar
      Move BreakInfo into own module · 7f2ce0d6
      Joachim Breitner authored and  Marge Bot's avatar Marge Bot committed
      while working on GHCi stuff, e.g. `GHC.Runtime.Eval.Types`, I observed a
      fair amount of modules being recompiled that I didn’t expect to depend
      on this, from byte code interpreters to linkers. Turns out that the
      rather simple `BreakInfo` type is all these modules need from the
      `GHC.Runtime.Eval.*` hierarchy, so by moving that into its own file we
      make the dependency tree wider and shallower, which is probably worth
    • Matthew Pickering's avatar
      Make sure paths are quoted in install Makefile · df016e4e
      Matthew Pickering authored and  Marge Bot's avatar Marge Bot committed
      Previously it would fail with this error:
      if [ -L wrappers/ghc ]; then echo "ghc is a symlink"; fi
      ghc is a symlink
      cp: target 'dir/bin/ghc' is not a directory
      make: *** [Makefile:197: install_wrappers] Error 1
      which is because the install path contains a space.
      Fixes #20506
  2. 13 Oct, 2021 12 commits
  3. 12 Oct, 2021 22 commits