StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge changes I7a102fa1,Ic3156e53 into main

Browse Source 
* changes:
  [cc:riscv64] change -munaligned-access to -mno-strict-align
  Fully document all the riscv64 flags.
This commit is contained in:
Treehugger Robot
2024-03-19 17:42:45 +00:00
committed by Gerrit Code Review
parent 3e8bd45638 0593537f58
commit 97409cd9a2
1 changed files with 19 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
cc/config/riscv64_device.go
View File

@@ -23,24 +23,36 @@ import (
var ( var (
riscv64Cflags = []string{ riscv64Cflags = []string{
// Help catch common 32/64-bit errors. // Help catch common 32/64-bit errors. (This is duplicated in all 64-bit
// architectures' cflags.)
"-Werror=implicit-function-declaration", "-Werror=implicit-function-declaration",
// This is already the driver's Android default, but duplicated here (and
// below) for ease of experimentation with additional extensions.
"-march=rv64gcv_zba_zbb_zbs", "-march=rv64gcv_zba_zbb_zbs",
"-munaligned-access", // TODO: move to driver (https://github.com/google/android-riscv64/issues/111)
// Until https://gitlab.com/qemu-project/qemu/-/issues/1976 is fixed... "-mno-strict-align",
// TODO: remove when qemu V works (https://gitlab.com/qemu-project/qemu/-/issues/1976)
// (Note that we'll probably want to wait for berberis to be good enough
// that most people don't care about qemu's V performance either!)
"-mno-implicit-float", "-mno-implicit-float",
// (https://github.com/google/android-riscv64/issues/124) // TODO: remove when clang default changed (https://github.com/google/android-riscv64/issues/124)
"-mllvm -jump-is-expensive=false", "-mllvm -jump-is-expensive=false",
} }
riscv64ArchVariantCflags = map[string][]string{} riscv64ArchVariantCflags = map[string][]string{}
riscv64Ldflags = []string{ riscv64Ldflags = []string{
// TODO: sysv hashes are the default for other architectures because gnu
// hashes weren't supported until api level 23, but riscv64 didn't exist
// back then, and could move today...
// https://android.googlesource.com/platform/bionic/+/main/android-changes-for-ndk-developers.md#gnu-hashes-availible-in-api-level-23
"-Wl,--hash-style=gnu", "-Wl,--hash-style=gnu",
// This is already the driver's Android default, but duplicated here (and
// above) for ease of experimentation with additional extensions.
"-march=rv64gcv_zba_zbb_zbs", "-march=rv64gcv_zba_zbb_zbs",
"-munaligned-access", // TODO: move to driver (https://github.com/google/android-riscv64/issues/111)
// We should change the default for this in clang, but for now... "-mno-strict-align",
// (https://github.com/google/android-riscv64/issues/124) // TODO: remove when clang default changed (https://github.com/google/android-riscv64/issues/124)
"-Wl,-mllvm -Wl,-jump-is-expensive=false", "-Wl,-mllvm -Wl,-jump-is-expensive=false",
} }