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

Merge "Enable CFI for riscv64" into main am: 5c2b3e2ea6

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2595746

Change-Id: I1cc89b91504b81ecd486032e65ea4345e9245bc2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Aditya Kumar
2023-07-28 06:26:42 +00:00
committed by Automerger Merge Worker
parent 1408bd0cbb 5c2b3e2ea6
commit f2886e1913
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cc/sanitize.go
View File

@@ -671,12 +671,6 @@ func (sanitize *sanitize) begin(ctx BaseModuleContext) {
s.Integer_overflow = nil s.Integer_overflow = nil
} }
// TODO(b/254713216): CFI doesn't work for riscv64 yet because LTO doesn't work.
if ctx.Arch().ArchType == android.Riscv64 {
s.Cfi = nil
s.Diag.Cfi = nil
}
// Disable CFI for musl // Disable CFI for musl
if ctx.toolchain().Musl() { if ctx.toolchain().Musl() {
s.Cfi = nil s.Cfi = nil