[PATCH] D117286: [msan] Reset shadow of byval before call

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 14 12:52:49 PST 2022


vitalybuka added a comment.

In D117286#3244655 <https://reviews.llvm.org/D117286#3244655>, @eugenis wrote:

> Is this necessary if we zero byval pointee shadow in the other patch?

without the patch:

1. test line 82 with byval -> reset and then copy shadow
2. test line 82 without byval -> copy shadow

with the patch:

1. test line 82 with byval -> reset shadow
2. test line 82 without byval -> reset shadow


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117286/new/

https://reviews.llvm.org/D117286



More information about the llvm-commits mailing list