Commits on Source (2)
-
Sebastian Graf authored
In #18793, we saw a compelling example which requires us to look at non-recursive let-bindings during arity analysis and unleash their arity types at use sites. After the refactoring in the previous patch, the needed change is quite simple and very local to `arityType`'s defn for non-recurisve `Let`. Apart from that, we had to get rid of the second item of `Note [Dealing with bottoms]`, which was entirely a safety measure and hindered optimistic fixed-point iteration. Fixes #18793. The following metric increases are all caused by this commit and a result of the fact that we just do more work now: Metric Increase: T3294 T12545 T12707
89d2a1b3 -
Sebastian Graf authored
We didn't seem to test these old tests at all, judging from their expected output.
67b8e742
Showing
- compiler/GHC/Core/Opt/Arity.hs 106 additions, 62 deletionscompiler/GHC/Core/Opt/Arity.hs
- testsuite/tests/arityanal/Main.hs 0 additions, 284 deletionstestsuite/tests/arityanal/Main.hs
- testsuite/tests/arityanal/Main.stderr 0 additions, 1874 deletionstestsuite/tests/arityanal/Main.stderr
- testsuite/tests/arityanal/Makefile 3 additions, 0 deletionstestsuite/tests/arityanal/Makefile
- testsuite/tests/arityanal/f0.stderr 0 additions, 12 deletionstestsuite/tests/arityanal/f0.stderr
- testsuite/tests/arityanal/f1.stderr 0 additions, 20 deletionstestsuite/tests/arityanal/f1.stderr
- testsuite/tests/arityanal/f10.stderr 0 additions, 27 deletionstestsuite/tests/arityanal/f10.stderr
- testsuite/tests/arityanal/f11.stderr 0 additions, 45 deletionstestsuite/tests/arityanal/f11.stderr
- testsuite/tests/arityanal/f12.stderr 0 additions, 8 deletionstestsuite/tests/arityanal/f12.stderr
- testsuite/tests/arityanal/f13.stderr 0 additions, 17 deletionstestsuite/tests/arityanal/f13.stderr
- testsuite/tests/arityanal/f14.stderr 0 additions, 26 deletionstestsuite/tests/arityanal/f14.stderr
- testsuite/tests/arityanal/f15.stderr 0 additions, 14 deletionstestsuite/tests/arityanal/f15.stderr
- testsuite/tests/arityanal/f2.stderr 0 additions, 18 deletionstestsuite/tests/arityanal/f2.stderr
- testsuite/tests/arityanal/f3.stderr 0 additions, 15 deletionstestsuite/tests/arityanal/f3.stderr
- testsuite/tests/arityanal/f4.stderr 0 additions, 22 deletionstestsuite/tests/arityanal/f4.stderr
- testsuite/tests/arityanal/f5.stderr 0 additions, 47 deletionstestsuite/tests/arityanal/f5.stderr
- testsuite/tests/arityanal/f6.stderr 0 additions, 16 deletionstestsuite/tests/arityanal/f6.stderr
- testsuite/tests/arityanal/f7.stderr 0 additions, 14 deletionstestsuite/tests/arityanal/f7.stderr
- testsuite/tests/arityanal/f8.stderr 0 additions, 20 deletionstestsuite/tests/arityanal/f8.stderr
- testsuite/tests/arityanal/f9.stderr 0 additions, 15 deletionstestsuite/tests/arityanal/f9.stderr
testsuite/tests/arityanal/Main.hs
deleted
100644 → 0
This diff is collapsed.
testsuite/tests/arityanal/f0.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f1.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f10.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f11.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f12.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f13.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f14.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f15.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f2.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f3.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f4.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f5.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f6.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f7.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f8.stderr
deleted
100644 → 0
testsuite/tests/arityanal/f9.stderr
deleted
100644 → 0