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

[DNM] Integrate various ARM branches #424

Closed
wants to merge 12 commits into from
Closed

Conversation

fw-immunant
Copy link
Contributor

Some of the commits here from other branches are stale and others are modified compared to their upstream branch, but this successfully runs a number of tests locally with MTE enforced. PR for CI only for now.

@fw-immunant fw-immunant changed the title Integrate various ARM branches [DNM] Integrate various ARM branches Oct 5, 2024
@fw-immunant fw-immunant force-pushed the fw/arm-integration branch 5 times, most recently from dc6d8e6 to 55d64ce Compare October 7, 2024 20:09
@fw-immunant fw-immunant changed the base branch from main to am/scrub_registers October 7, 2024 20:21
@rinon rinon force-pushed the am/scrub_registers branch from e12ee3d to f986f93 Compare October 7, 2024 22:36
@rinon rinon force-pushed the am/scrub_registers branch from d9fdc1a to 6372a16 Compare October 11, 2024 21:11
@fw-immunant fw-immunant changed the base branch from am/scrub_registers to main October 21, 2024 19:14
@fw-immunant fw-immunant force-pushed the fw/arm-integration branch 7 times, most recently from d0067ed to b509501 Compare October 25, 2024 18:31
ayrtonm and others added 10 commits October 28, 2024 12:40
Without this CTest only prints tests that fail so MTE check failures logged by
QEMU won't show up unless the test fails for another reason.
this must apply to all code including PartitionAlloc. as such, we don't need to inject these specifically for libia2 and dependencies.

while we're at it, ask specifically for armc8.5-a, which is where MTE and BTI support were added
rpath-link is needed for the link step, rpath is needed for runtime

this may conflict with what CMake wants to do with rpaths, but seems to work for now
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