[PATCH] D92632: [dfsan] Add a test case of storing zero
stephan.yichao.zhao via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 4 08:36:54 PST 2020
stephan.yichao.zhao marked an inline comment as done.
stephan.yichao.zhao added inline comments.
================
Comment at: llvm/test/Instrumentation/DataFlowSanitizer/store.ll:163
+define void @store_zero(i32* %p) {
+ ; NO_COMBINE_PTR_LABEL: store i64 0, i64* {{.*}}, align 2
+ store i32 0, i32* %p
----------------
morehouse wrote:
> Why is it storing i64 shadow?
It is testing this branch:
https://github.com/llvm/llvm-project/blob/master/llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp#L1465
The shadow of (i32 0) is (i64 0).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92632/new/
https://reviews.llvm.org/D92632
More information about the llvm-commits
mailing list