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

Basic debugging information support #1331

Merged
merged 21 commits into from
Dec 19, 2011
Merged

Basic debugging information support #1331

merged 21 commits into from
Dec 19, 2011

Conversation

jdm
Copy link
Contributor

@jdm jdm commented Dec 19, 2011

nmatsakis expressed interest in integrating even basic debugging support, and that's what this provides. The -g flag will produce source line correspondence output, allowing for stepping and breakpoints inside gdb. There's also an --xg (extra debugging) flag behind which the rest of the debugging support lies unfinished (ie. can't build rustc completely yet, but works on toy programs).

Some stats:

godot:build jdm$ time x86_64-apple-darwin/stage2/bin/rustc --target=x86_64-apple-darwin  -o x86_64-apple-darwin/stage2/lib/rustc/x86_64-apple-darwin/bin/rustc ../src/comp/rustc.rc
warning: no debug symbols in executable (-arch x86_64)

real    0m40.174s
user    0m39.633s
sys 0m0.708s

godot:build jdm$ time x86_64-apple-darwin/stage2/bin/rustc -g --target=x86_64-apple-darwin  -o x86_64-apple-darwin/stage2/lib/rustc/x86_64-apple-darwin/bin/rustc ../src/comp/rustc.rc

real    1m50.058s
user    1m49.456s
sys 0m1.125s

Requires the llvm patch that's sitting in brson's repository to allow rustc to finish building.

@marijnh
Copy link
Contributor

marijnh commented Dec 19, 2011

Please run make check at least to get rid of overlong lines and trailing whitespace.

@jdm
Copy link
Contributor Author

jdm commented Dec 19, 2011

The extra llvm patch is no longer required as of 6637340.

@marijnh marijnh merged commit 6637340 into rust-lang:master Dec 19, 2011
@marijnh
Copy link
Contributor

marijnh commented Dec 19, 2011

Passes now. Merged.

@nikomatsakis
Copy link
Contributor

I was just going to pull this, but I see Marijn beat me to it. Can't wait to try it out!

@kud1ing kud1ing mentioned this pull request Dec 20, 2011
celinval pushed a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
* Resolved link formatting issue.

* Fixed escaping of brackets.

* Fixed intra dock links.

* Fixed broken link to rustc_middle::ty::layout::LayoutCx.

* More hyperlink formatting warning fixed.

* Fixed typo in struct. Mismatched curly braces.

* fixed links and brackets.

* Added gh action to build docs automatically.

* Added gh action to build docs automatically.

* Moved docs check to regressions script.
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.

None yet

3 participants