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 needed for 'x86_64-fortanix-unknown-sgx' nightly target. #62

Closed
wants to merge 1 commit into from
Closed

Changes needed for 'x86_64-fortanix-unknown-sgx' nightly target. #62

wants to merge 1 commit into from

Conversation

AdrianCX
Copy link

This is an update of PR to another branch:

Original review and comments: #57 (comment)

Code is guarded via defines to enable only if 'RUST_SGX' is present.

Main logic is in libunwind/src/AddressSpace.hpp
We use 6 symbols to figure out where eh_frame / eh_frame_hdr is at runtime when loaded in an SGX enclave. (EH symbols + IMAGE base)
These are set by 'fortanix-sgx-tools'.

As notes:

  • Target above at the moment uses a pre-compiled libunwind.a from forked repo.
  • Goal of these changes is to use official llvm with patch.
  • Changes in rust-lang to use this are planned if/when this is accepted.
  • Ticket: Update libunwind fortanix/rust-sgx#174
  • Original ported changes: llvm/llvm-project@release/5.x...fortanix:release/5.x

Code is guarded via defines to enable only if 'RUST_SGX' is present.

Main logic is in libunwind/src/AddressSpace.hpp
We use 6 symbols to figure out where eh_frame / eh_frame_hdr is at runtime when loaded in an SGX enclave. (EH symbols + IMAGE base)
These are set by 'fortanix-sgx-tools'.

As notes:
- Target above at the moment uses a pre-compiled libunwind.a from forked repo.
- Goal of these changes is to use official llvm with patch.
- Changes in rust-lang to use this are planned if/when this is accepted.
- Ticket: fortanix/rust-sgx#174
- Original ported changes: llvm/llvm-project@release/5.x...fortanix:release/5.x
@AdrianCX AdrianCX closed this May 26, 2020
vext01 added a commit to vext01/llvm-project that referenced this pull request Apr 27, 2023
62: Add the yk-linkage llvm pass. r=ltratt a=vext01



Co-authored-by: Edd Barrett <vext01@gmail.com>
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.

1 participant