Enna1 added a comment.
Herald added subscribers: yaneury, supersymetrie, Chia-hungDuan, pengfei.
Herald added a project: All.
Hi, I'm curious about why hwasan requires PIE. Is there any documents mentioned
this? Thanks!
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.or
This revision was automatically updated to reflect the committed changes.
Closed by commit rC328361: [HWASan] Port HWASan to Linux x86-64 (clang)
(authored by alekseyshl, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44745?vs=139513&id=139640#toc
Repository:
rC Clang
ht
eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.
OK, sure, If you feel so strongly about this.
Repository:
rC Clang
https://reviews.llvm.org/D44745
___
cfe-commits mailing list
cfe-commits@
alekseyshl updated this revision to Diff 139513.
alekseyshl added a comment.
- Revert triple change in asan.cc test
Repository:
rC Clang
https://reviews.llvm.org/D44745
Files:
lib/Driver/SanitizerArgs.cpp
lib/Driver/ToolChains/Linux.cpp
test/Driver/Inputs/resource_dir/lib/linux/libclan
alekseyshl added inline comments.
Comment at: test/Driver/asan.c:12
-// RUN: %clang -O2 -target aarch64-unknown-linux -fsanitize=hwaddress %s -S
-emit-llvm -o - | FileCheck %s --check-prefix=CHECK-HWASAN
-// RUN: %clang -O3 -target aarch64-unknown-linux -fsanitize=hwaddress %s -
eugenis added inline comments.
Comment at: test/Driver/asan.c:12
-// RUN: %clang -O2 -target aarch64-unknown-linux -fsanitize=hwaddress %s -S
-emit-llvm -o - | FileCheck %s --check-prefix=CHECK-HWASAN
-// RUN: %clang -O3 -target aarch64-unknown-linux -fsanitize=hwaddress %s -S
alekseyshl updated this revision to Diff 139469.
alekseyshl added a comment.
- Add tests to verify that HWASan requires "-pie".
Repository:
rC Clang
https://reviews.llvm.org/D44745
Files:
lib/Driver/SanitizerArgs.cpp
lib/Driver/ToolChains/Linux.cpp
test/Driver/Inputs/resource_dir/lib/l
alekseyshl added inline comments.
Comment at: test/Driver/asan.c:12
-// RUN: %clang -O2 -target aarch64-unknown-linux -fsanitize=hwaddress %s -S
-emit-llvm -o - | FileCheck %s --check-prefix=CHECK-HWASAN
-// RUN: %clang -O3 -target aarch64-unknown-linux -fsanitize=hwaddress %s -
eugenis added inline comments.
Comment at: lib/Driver/SanitizerArgs.cpp:33
NotAllowedWithMinimalRuntime = Vptr,
- RequiresPIE = DataFlow | Scudo,
+ RequiresPIE = DataFlow | HWAddress | Scudo,
NeedsUnwindTables = Address | HWAddress | Thread | Memory | DataFlow,
--
alekseyshl created this revision.
alekseyshl added a reviewer: eugenis.
Herald added a subscriber: cryptoad.
Porting HWASan to Linux x86-64, the third of the three patches, clang part.
Repository:
rC Clang
https://reviews.llvm.org/D44745
Files:
lib/Driver/SanitizerArgs.cpp
lib/Driver/Too
10 matches
Mail list logo