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

Unwinding for ARM option2 #5432

Merged
merged 4 commits into from
Mar 20, 2013
Merged

Conversation

ILyoan
Copy link
Contributor

@ILyoan ILyoan commented Mar 19, 2013

Partial Fix for #5265

  • Enabling LLVM ARM ehabi option.
  • Add ARM debug information manually for ccall.s
  • Compile object file using Android-NDK.

Current LLVM trunk version can generate ARM debug information for assembly files but it is incomplete for object files. Unwinding on ARM can be done with LLVM trunk(the LLVM submodule of rust has problem on generating ARM debug information). See #5368

The Android-NDK detour(0f89eab) can be removed after LLVM has complete feature of generating ARM debug information for object file.

@ILyoan
Copy link
Contributor Author

ILyoan commented Mar 19, 2013

I believe this patch is harmless for other platforms than android.
With this patch and upgraded LLVM(ongoing in #5368), unwinding tests pass.

bors added a commit that referenced this pull request Mar 20, 2013
Partial Fix for #5265

- Enabling LLVM ARM ehabi option.
- Add ARM debug information manually for ccall.s
- Compile object file using Android-NDK.

Current LLVM trunk version can generate ARM debug information for assembly files but it is incomplete for object files. Unwinding on ARM can be done with LLVM trunk(the LLVM submodule of rust has problem on generating ARM debug information). See #5368

The Android-NDK detour(0f89eab) can be removed after LLVM has complete feature of generating ARM debug information for object file.
@bors bors closed this Mar 20, 2013
@bors bors merged commit c1cacc3 into rust-lang:incoming Mar 20, 2013
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