This revision was automatically updated to reflect the committed changes.
Closed by commit rL320232: Hardware-assisted AddressSanitizer (clang part).
(authored by eugenis).
Changed prior to commit:
https://reviews.llvm.org/D40936?vs=126060=126252#toc
Repository:
rL LLVM
This revision was automatically updated to reflect the committed changes.
Closed by commit rC320232: Hardware-assisted AddressSanitizer (clang part).
(authored by eugenis).
Changed prior to commit:
https://reviews.llvm.org/D40936?vs=126060=126251#toc
Repository:
rC Clang
eugenis updated this revision to Diff 126060.
eugenis added a comment.
clang-format
https://reviews.llvm.org/D40936
Files:
clang/include/clang/Basic/Sanitizers.def
clang/include/clang/Driver/SanitizerArgs.h
clang/lib/CodeGen/BackendUtil.cpp
clang/lib/CodeGen/CGDeclCXX.cpp
eugenis added inline comments.
Comment at: clang/include/clang/Basic/Sanitizers.def:47
+SANITIZER("hwaddress", HWAddress)
+
alekseyshl wrote:
> Why didn't we follow KASan style and name it "hw-address"?
A matter of taste, I guess.
I don't want the attribute to
alekseyshl accepted this revision.
alekseyshl added inline comments.
Comment at: clang/include/clang/Basic/Sanitizers.def:47
+SANITIZER("hwaddress", HWAddress)
+
Why didn't we follow KASan style and name it "hw-address"?
Comment at:
kcc accepted this revision.
kcc added a comment.
This revision is now accepted and ready to land.
LGTM
please give at least Aleksey a chance to review as well.
https://reviews.llvm.org/D40936
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
eugenis created this revision.
Herald added subscribers: javed.absar, srhines.
Driver, frontend and LLVM codegen for HWASan.
A clone of ASan, basically.
https://reviews.llvm.org/D40936
Files:
clang/include/clang/Basic/Sanitizers.def
clang/include/clang/Driver/SanitizerArgs.h