Skip to content

Update DWARF tests to Ubuntu 24.04 #11077

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

Merged
merged 1 commit into from
Jun 19, 2025

Conversation

philipc
Copy link
Contributor

@philipc philipc commented Jun 19, 2025

The gdb version in Ubuntu 24.04 fails on our synthetic DWARF because:

  1. the synthetic DWARF unit has no DW_AT_language
  2. synthetic types are referenced from a unit that does have DW_AT_language
  3. this triggers https://sourceware.org/bugzilla/show_bug.cgi?id=32431

Workaround this issue by specifying DW_LANG_C11. This may not be exactly the right meaning, but it should be safe.

Closes #9731

I would like this change because my development system has this gdb version.

The gdb version in Ubuntu 24.04 fails on our synthetic DWARF because:

1. the synthetic DWARF unit has no DW_AT_language
2. synthetic types are referenced from a unit that does have DW_AT_language
3. this triggers https://sourceware.org/bugzilla/show_bug.cgi?id=32431

Workaround this issue by specifying DW_LANG_C11. This may not be
exactly the right meaning, but it should be safe.
@philipc philipc requested review from a team as code owners June 19, 2025 12:55
@philipc philipc requested review from abrown and removed request for a team June 19, 2025 12:55
Copy link
Member

@abrown abrown left a comment

Choose a reason for hiding this comment

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

This all seems reasonable to me as it updates tools to more recent versions. I'm no DWARF expert, though, so I'll leave this open for a bit in case @alexcrichton or @fitzgen have any stronger opinions about the C11 decision.

@alexcrichton alexcrichton added this pull request to the merge queue Jun 19, 2025
Merged via the queue into bytecodealliance:main with commit 329ae91 Jun 19, 2025
41 checks passed
@philipc philipc deleted the ubuntu-24.04 branch June 19, 2025 22:05
@fitzgen
Copy link
Member

fitzgen commented Jun 20, 2025

Thanks @philipc!

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.

Update DWARF tests in CI to Ubuntu 24.04
4 participants