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

Re-factor SGX directory structure #588

Merged
merged 3 commits into from
Feb 16, 2022
Merged

Re-factor SGX directory structure #588

merged 3 commits into from
Feb 16, 2022

Conversation

csegarragonz
Copy link
Collaborator

@csegarragonz csegarragonz commented Feb 15, 2022

Changelog:

  • Re-factor {src,include,tests/tests}/sgx to {src,include,tests/tests}/enclave.
  • Remove attestation.cpp and attestation.h as they were no longer used.
  • Sub-divide {src,include}/enclave in {src,include}/enclave/{inside,outside} for code that runs inside the enclave and outside, with separate CMakeLists.txt.

@csegarragonz csegarragonz self-assigned this Feb 15, 2022
@csegarragonz csegarragonz force-pushed the sgx-directories branch 3 times, most recently from 67be0a3 to 5d72da6 Compare February 15, 2022 13:47
@@ -74,9 +73,6 @@ class SGXWAMRWasmModule final : public WasmModule

uint8_t* getMemoryBase() override;

// TODO: Move in gs/fs
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This class member was defined in attestation.h and was not used.

@csegarragonz csegarragonz marked this pull request as ready for review February 15, 2022 13:48
@csegarragonz csegarragonz merged commit 96ad76e into main Feb 16, 2022
@csegarragonz csegarragonz deleted the sgx-directories branch February 16, 2022 08:59
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

2 participants