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

Move PIE from submodule into MAMBO repository #121

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Move PIE from submodule into MAMBO repository #121

merged 2 commits into from
Jun 6, 2024

Conversation

IgWod
Copy link
Collaborator

@IgWod IgWod commented May 21, 2024

This is something still very much to be discussed, but I think the current sentiment is to merge PIE and MAMBO together into a single repo. This has a number of benefits:

  1. Simplifies process of merging new instructions into MAMBO
  2. Reduces a possibility of MAMBO and PIE getting out of sync
  3. Simplifies repo management and access rights

I don't see any potential downside of such move, but I'm very much opened for comments.

Once that's done we need to port following PRs into this repo:

  1. Arm v8.2 pie#8
  2. A64: Add support for ARMv9.0 pie#9

@IgWod IgWod requested a review from jkressel May 21, 2024 14:56
@IgWod IgWod self-assigned this May 21, 2024
@IgWod IgWod marked this pull request as ready for review May 30, 2024 14:07
@ijan1
Copy link

ijan1 commented Jun 6, 2024

Might be worth putting in the path to https://github.com/beehive-lab/pie in a commit message for archival purposes, otherwise LGTM.

@jkressel jkressel merged commit ca06293 into master Jun 6, 2024
3 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