Skip to content

Commit 9a8f294

Browse files
frobtechtru
authored andcommitted
[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 b6089c6 commit 9a8f294

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
@@ -447,13 +447,13 @@ SanitizerMask Fuchsia::getDefaultSanitizers() const {
447447
SanitizerMask Res;
448448
switch (getTriple().getArch()) {
449449
case llvm::Triple::aarch64:
450+
case llvm::Triple::riscv64:
450451
Res |= SanitizerKind::ShadowCallStack;
451452
break;
452453
case llvm::Triple::x86_64:
453454
Res |= SanitizerKind::SafeStack;
454455
break;
455456
default:
456-
// TODO: Enable SafeStack on RISC-V once tested.
457457
break;
458458
}
459459
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)