Skip to content

Commit f08d86f

Browse files
committed
[RISCV] Default to -fsanitize=shadow-call-stack for Fuchsia
The ShadowCallStack is the preferred and default ABI for Fuchsia. Reviewed By: phosek Differential Revision: https://reviews.llvm.org/D143357
1 parent 8dac97e commit f08d86f

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

clang/lib/Driver/ToolChains/Fuchsia.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -466,13 +466,13 @@ SanitizerMask Fuchsia::getDefaultSanitizers() const {
466466
SanitizerMask Res;
467467
switch (getTriple().getArch()) {
468468
case llvm::Triple::aarch64:
469+
case llvm::Triple::riscv64:
469470
Res |= SanitizerKind::ShadowCallStack;
470471
break;
471472
case llvm::Triple::x86_64:
472473
Res |= SanitizerKind::SafeStack;
473474
break;
474475
default:
475-
// TODO: Enable SafeStack on RISC-V once tested.
476476
break;
477477
}
478478
return Res;

clang/test/Driver/fuchsia.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
// CHECK: "-isysroot" "[[SYSROOT:[^"]+]]"
3232
// CHECK: "-internal-externc-isystem" "[[SYSROOT]]{{/|\\\\}}include"
3333
// CHECK-AARCH64: "-fsanitize=shadow-call-stack"
34+
// CHECK-RISCV64: "-fsanitize=shadow-call-stack"
3435
// CHECK-X86_64: "-fsanitize=safe-stack"
3536
// CHECK: "-stack-protector" "2"
3637
// CHECK-AARCH64: "-target-feature" "+outline-atomics"

0 commit comments

Comments
 (0)