Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ghc: compile with DWARF support by default #68362

Merged
merged 1 commit into from Sep 10, 2019

Conversation

domenkozar
Copy link
Member

@domenkozar domenkozar commented Sep 9, 2019

This allows our GHCs to build programs with DWARF debug information
when -g is passed, see https://ghc.haskell.org/trac/ghc/wiki/DWARF.

Compiling with debug symbols is off by default until GHC ticket #15960
(Using -g causes differences in generated core) is fixed.

This differs with #52255 which makes the whole support optional and instead exposes another package set, which was brought up as problematic.

@domenkozar
Copy link
Member Author

CI failure doesn't come from this PR.

@adisbladis adisbladis removed their request for review September 9, 2019 16:35
This allows our GHCs to build programs with DWARF debug information
when -g is passed, see https://ghc.haskell.org/trac/ghc/wiki/DWARF.

Compiling with debug symbols is off by default until GHC ticket NixOS#15960
(Using -g causes differences in generated core) is fixed.
@domenkozar
Copy link
Member Author

I hoped this would go to 19.09 as it's really a small change, but crucial for Haskell in production.

@peti
Copy link
Member

peti commented Sep 10, 2019

No problem, this change can still make it into 19.09. I'll run the necessary test builds, and then it will be part of my Haskell Infrastructure sync.

@peti peti merged commit 6996d5c into NixOS:haskell-updates Sep 10, 2019
@domenkozar
Copy link
Member Author

Thank you @peti <3

@peti
Copy link
Member

peti commented Sep 18, 2019

This change broke ghc`s ability to link executables. The compiler is pretty much unusable in that state: #69023. Furthermore, it broke the compiler on Windows: 2021578.

@nh2
Copy link
Contributor

nh2 commented Apr 10, 2020

For those scrolling down here, this PR was reverted by PR #69032.

For anybody who wants to re-attempt it, please also update my still-open original PR #52255.

Relevant for GHC 8.10: #69552

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/ghc-with-libdw-support/6654/2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants