<div dir="ltr"><div dir="ltr">Thanks for bringing this up.  I've merged the PR and uploaded Happy 1.19.10 to Hackage.  Can someone else look at steps 3-5?</div><div dir="ltr"><br></div><div>Cheers</div><div>Simon<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 8 May 2019 at 09:51, Vladislav Zavialov <<a href="mailto:vladislav@serokell.io">vladislav@serokell.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;">Hello ghc-devs,<div><br></div><div>This February I did some changes to the parser that require higher rank types support in ‘happy’. Unfortunately, as I discovered, happy’s --coerce option is severely broken in the presence of higher rank types, so I had to disable it. My benchmarks have shown a 10% slowdown from disabling --coerce (<a href="https://gist.github.com/int-index/38af0c5dd801088dc1de59eca4e55df4" target="_blank">https://gist.github.com/int-index/38af0c5dd801088dc1de59eca4e55df4</a>).</div><div><br></div><div>Alongside my changes I submitted a pull request to happy which fixes the issue (<a href="https://github.com/simonmar/happy/pull/134" target="_blank">https://github.com/simonmar/happy/pull/134</a>), in the hope that it would get merged, released, and I could re-enable --coerce in GHC ‘happy' configuration.</div><div><br></div><div>Unfortunately, my patch has been ignored to this day (for 3 months now), and the performance regression reached 8.8-alpha. We need to act swiftly if we want to avoid a performance regression in the actual release. Here’s what needs to be done:</div><div><br></div><div>1. Merge <a href="https://github.com/simonmar/happy/pull/134" target="_blank">https://github.com/simonmar/happy/pull/134</a></div><div>2. Release a new ‘happy’</div><div>3. (Optional) Specify in GHC’s build system that it builds only with the latest 'happy' release</div><div>4. Restore the --coerce option in GHC’s build system ‘happy’ configuration</div><div>5. Backport it to the ghc-8.8 branch</div><div><br></div><div>I have no access to do 1 & 2, I believe Simon Marlow does. I’d appreciate if someone took care of 3, currently the build system does not install ‘happy’ and assumes a system-wide installation without checking its version. This means that users of all but the newly released version will encounter obscure error messages. We need a version check. Then I will do 4, as planned, and create a merge request for 5.</div><div><br></div><div>All the best,</div><div>- Vladislav</div></div>_______________________________________________<br>
ghc-devs mailing list<br>
<a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a><br>
<a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" rel="noreferrer" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs</a><br>
</blockquote></div></div>