summaryrefslogtreecommitdiff
path: root/compiler
diff options
context:
space:
mode:
authorAlan Zimmerman <alan.zimm@gmail.com>2020-10-28 23:09:48 +0000
committerAlan Zimmerman <alan.zimm@gmail.com>2020-11-02 08:23:42 +0000
commit791cb337549df002ff4323299e6baf825fe994f4 (patch)
treeba29187925df87daa5fcfa3c10a94179c720676a /compiler
parente63db32c7eb089985a1a7279a0a886a32d70ac0e (diff)
downloadhaskell-wip/T18888.tar.gz
Restrict Linear arrow %1 to exactly literal 1 onlywip/T18888
This disallows `a %001 -> b`, and makes sure the type literal is printed from its SourceText so it is clear why. Closes #18888
Diffstat (limited to 'compiler')
-rw-r--r--compiler/GHC/Hs/Type.hs4
-rw-r--r--compiler/GHC/Parser/PostProcess.hs3
2 files changed, 4 insertions, 3 deletions
diff --git a/compiler/GHC/Hs/Type.hs b/compiler/GHC/Hs/Type.hs
index 1845f060ed..1ae23c779d 100644
--- a/compiler/GHC/Hs/Type.hs
+++ b/compiler/GHC/Hs/Type.hs
@@ -1982,8 +1982,8 @@ ppr_fun_ty mult ty1 ty2
--------------------------
ppr_tylit :: HsTyLit -> SDoc
-ppr_tylit (HsNumTy _ i) = integer i
-ppr_tylit (HsStrTy _ s) = text (show s)
+ppr_tylit (HsNumTy source i) = pprWithSourceText source (integer i)
+ppr_tylit (HsStrTy source s) = pprWithSourceText source (text (show s))
-- | @'hsTypeNeedsParens' p t@ returns 'True' if the type @t@ needs parentheses
diff --git a/compiler/GHC/Parser/PostProcess.hs b/compiler/GHC/Parser/PostProcess.hs
index 84aa3e09bc..e95abd6597 100644
--- a/compiler/GHC/Parser/PostProcess.hs
+++ b/compiler/GHC/Parser/PostProcess.hs
@@ -2620,7 +2620,8 @@ mkLHsOpTy x op y =
in L loc (mkHsOpTy x op y)
mkMultTy :: IsUnicodeSyntax -> Located Token -> LHsType GhcPs -> (HsArrow GhcPs, AddAnn)
-mkMultTy u tok t@(L _ (HsTyLit _ (HsNumTy _ 1)))
+mkMultTy u tok t@(L _ (HsTyLit _ (HsNumTy (SourceText "1") 1)))
+ -- See #18888 for the use of (SourceText "1") above
= (HsLinearArrow u, AddAnn AnnPercentOne (combineLocs tok t))
mkMultTy u tok t = (HsExplicitMult u t, AddAnn AnnPercent (getLoc tok))