-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add -m[no-][scalar|vector]-strict-align #50
base: main
Are you sure you want to change the base?
Conversation
Ref: |
CC @preames |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I shared my opinion on a related proposal here: riscv-non-isa/riscv-c-api-doc#73 (comment)
I don't personally plan to use these flags, so I'm OK if it remains GCC-specific behavior, but I won't fight it if others in the community think it's actually useful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but has a dependency for #49.
…. Alias -m[no-]strict-align to scalar. __riscv_misaligned_fast will be set based on -mno-scalar-strict-align or -mno-strict-align. This matches the direction gcc is proposing. See riscv-non-isa/riscv-c-api-doc#73 riscv-non-isa/riscv-toolchain-conventions#49 riscv-non-isa/riscv-toolchain-conventions#50 riscv-non-isa/riscv-c-api-doc#80
090aedc
to
79008be
Compare
b664b96
to
174a844
Compare
Changes:
|
Add two new option `-m[no-]scalar-strict-align` for alias of `-m[no-]-strict-align` and `-m[no-][vector]-strict-align` for vector misaligned access.
174a844
to
8b45411
Compare
Changes:
|
Add two new option
-m[no-]scalar-strict-align
for alias of-m[no-]-strict-align
and-m[no-][vector]-strict-align
for vector misaligned access.