[cpullvm][Multilib] Add aligned AArch64 variants#448
Open
Jonathon Penix (jonathonpenix) wants to merge 2 commits into
Open
Conversation
…ants that currently build with unaligned access (qualcomm#389) Requiring `-munaligned-access` for multilib should provide better build-time safeguards if people try using `-mno-unaligned-access` when they truly need it disabled. If people require an aligned variant, we'll consider adding one in the future. And, if people want to mix and match (build with `-mno-unaligned-access` but match against a library built with `-munaligned-access`) they can still override multilib and point at the library variant manually. At the same time, expicitly specify `-munaligned-access` for these configs in the compile flags. This should be NFC, but will hopefully make it more clear what our intent for these variants is and safeguard us from any unforseen changes in defaults, etc. Note that this also shouldn't impact our musl configs where we specify `-mstrict-align`/`-mno-unaligned-access` via `EXTRA_MUSL-EMBEDDED_CFLAGS`--those are used when building musl *only* and those flags are placed after the generic `COMPILE_FLAGS` so will override them. As mentioned above, this should resolve qualcomm#388 for now until we want to support aligned variants. (Backported from 13fce8b) Signed-off-by: Jonathon Penix <jpenix@qti.qualcomm.com>
Add aligned alternatives of the following variants: - aarch64a_tlsie - aarch64a_pacret_bkey_bti_tlsie - aarch64a_soft_nofp_tlsie - aarch64a_soft_nofp_pacret_bti So far these are the only variants where folks have needed unaligned variants, so we're just going to duplicate these for now. See qualcomm#388 for additional context around our Arm/AArch64 variant alignment situation. Signed-off-by: Jonathon Penix <jpenix@qti.qualcomm.com>
Jonathon Penix (jonathonpenix)
force-pushed
the
pr/22_aligned_aarch64_variants
branch
from
8063348 to
7340d09
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add aligned alternatives of the following variants:
So far these are the only variants where folks have needed unaligned
variants, so we're just going to duplicate these for now.
See #388 for additional
context around our Arm/AArch64 variant alignment situation.
=========
Stacked on #447