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

changes to adapt to compressed line table format #526

Merged
merged 3 commits into from
Mar 26, 2024

Conversation

vtjnash
Copy link
Contributor

@vtjnash vtjnash commented Jan 3, 2024

This contains some updates needed to keep working with the optimized debuginfo format (JuliaLang/julia#52415).

Also needs JuliaDebug/JuliaInterpreter.jl#606

@vtjnash vtjnash marked this pull request as draft January 3, 2024 20:00
@vtjnash vtjnash force-pushed the jn/invert-line-table branch from d7379f1 to 71efc78 Compare January 5, 2024 16:28
@vtjnash vtjnash force-pushed the jn/invert-line-table branch from 71efc78 to 895eea4 Compare February 22, 2024 20:50
@vtjnash vtjnash force-pushed the jn/invert-line-table branch from 895eea4 to 8d49de6 Compare March 7, 2024 21:14
@vtjnash vtjnash changed the title WIP changes to adapt to compressed line table format changes to adapt to compressed line table format Mar 7, 2024
@vtjnash vtjnash marked this pull request as ready for review March 7, 2024 21:14
@codecov-commenter
Copy link

codecov-commenter commented Mar 7, 2024

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (f3dd6cd) to head (f99d97c).

Files Patch % Lines
src/reflection.jl 0.00% 14 Missing ⚠️
src/interpreter.jl 0.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master    #526   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files           9       9           
  Lines        1541    1545    +4     
======================================
- Misses       1541    1545    +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

src/reflection.jl Outdated Show resolved Hide resolved
TypedSyntax/src/node.jl Outdated Show resolved Hide resolved
TypedSyntax/src/node.jl Outdated Show resolved Hide resolved
src/reflection.jl Outdated Show resolved Hide resolved
@aviatesk aviatesk force-pushed the jn/invert-line-table branch from 46bb524 to 96020bd Compare March 23, 2024 11:50
@aviatesk aviatesk force-pushed the jn/invert-line-table branch 2 times, most recently from bfe840f to f99d97c Compare March 26, 2024 05:08
@aviatesk
Copy link
Member

TypedSyntax test is failing badly...

@aviatesk aviatesk force-pushed the jn/invert-line-table branch from f99d97c to 3c3759e Compare March 26, 2024 10:16
@aviatesk
Copy link
Member

I adjusted the TypedSyntax update so now the number of failing tests matches what we're seeing with v1.11-nightly. I'm going to push ahead with this as it stands to keep working on other packages. I aim to implement more conservative and safer changes in the near future.

@aviatesk
Copy link
Member

Ok, Cthulhu CI is green on nightly. I will go ahead and merge.

@aviatesk aviatesk merged commit 09c4fed into JuliaDebug:master Mar 26, 2024
18 of 29 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.

3 participants