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 am: f2886e1913

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

Change-Id: I99ee9f64a53b231238100f9d1a95f49138fd6178
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Aditya Kumar
2023-07-28 07:13:36 +00:00
committed by Automerger Merge Worker
parent acefeef15a f2886e1913
commit 3dcc091676
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
}
// 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
if ctx.toolchain().Musl() {
s.Cfi = nil