Skip to content

HDB+GHC session for HLS and debug#308

Merged
Saizan merged 2 commits into
masterfrom
hdb-ghc-debug-on-abstract
May 21, 2026
Merged

HDB+GHC session for HLS and debug#308
Saizan merged 2 commits into
masterfrom
hdb-ghc-debug-on-abstract

Conversation

@Saizan
Copy link
Copy Markdown
Collaborator

@Saizan Saizan commented May 4, 2026

See DEBUG.md for required setup and limitations

@alt-romes
Copy link
Copy Markdown
Collaborator

I should also land my changes that introduce the view as HTML DebugView instance for ModuleGraph :)

@Saizan Saizan force-pushed the hdb-ghc-debug-on-abstract branch 3 times, most recently from 0b8044b to e06a2b5 Compare May 7, 2026 08:41
@Saizan Saizan marked this pull request as ready for review May 7, 2026 08:42
@Saizan Saizan force-pushed the hdb-ghc-debug-on-abstract branch 2 times, most recently from 5d50081 to c99b8eb Compare May 8, 2026 11:57
@Saizan Saizan requested a review from alt-romes May 8, 2026 13:18
Copy link
Copy Markdown
Collaborator

@alt-romes alt-romes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a few tweaks

Comment thread DEBUG.md Outdated
Comment thread DEBUG.md Outdated
Comment thread DEBUG.md Outdated
Comment thread DEBUG.md
Comment on lines +15 to +17
- Setup the final debuggee with a cradle that won't use package dbs setup by HIE.
- See Limitations
- See ./scripts/cabal-repl-bios.sh for a script that can be used as a bios cradle for this, uses cabal repl.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand this too well. I thought setting the hie.yaml to debug-with-ghc-bios to be sufficient? does this mean we have to do something else?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I was getting segfaults when the debugee's debugee was using a normal cradle and using the above script with a bios cradle helped.

So I suppose you can start without and look into this if you get a segfault while the units are getting setup. At some point we'll actually solve this.

Copy link
Copy Markdown
Collaborator

@alt-romes alt-romes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm (please squash into independent commits only)

Comment thread DEBUG.md
- See https://gitlab.haskell.org/trac-Saizan/ghc/-/commits/debug-9.14
- Gone through early stages of prep and build so `./ghc/_build/stageBoot/bin/` is populated.
- Installed ghc of same version and flavour as GHC source tree being debugged.
- Installed hdb with executable-dynamic: True.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this hdb have to be compiled by the same ghc as above? If yes say explicitly

@Saizan Saizan force-pushed the hdb-ghc-debug-on-abstract branch from 533c6f9 to 3a0ea80 Compare May 21, 2026 16:13
@Saizan Saizan merged commit 3a0ea80 into master May 21, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants