[PATCH] D116634: enable msan-eager-checks with -fsanitize-memory-param-retval.

2022-01-07 Thread Kevin Athey via Phabricator via cfe-commits
kda updated this revision to Diff 398310. kda added a comment. add 3 parameter constructor for MemorySanitizerOptions Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116634/new/ https://reviews.llvm.org/D116634 Files:

[PATCH] D116634: enable msan-eager-checks with -fsanitize-memory-param-retval.

2022-01-07 Thread Kevin Athey via Phabricator via cfe-commits
kda added inline comments. Comment at: llvm/include/llvm/Transforms/Instrumentation/MemorySanitizer.h:28 bool Recover; + bool EagerChecks; }; vitalybuka wrote: > maybe we should use CheckParamRetVal something here? > How about EagerChecksRequested?

[PATCH] D116634: enable msan-eager-checks with -fsanitize-memory-param-retval.

2022-01-07 Thread Kevin Athey via Phabricator via cfe-commits
kda updated this revision to Diff 398303. kda marked 2 inline comments as done. kda added a comment. Reworked for comments and to be llvm only. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116634/new/ https://reviews.llvm.org/D116634 Files:

[PATCH] D116634: enable msan-eager-checks with -fsanitize-memory-param-retval.

2022-01-04 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added a comment. this patch depends on D116633 , but D116633 is useless without this one a better patch order is : 1. llvm 2. clang 3. compiler-rt (but if it's just test which uses clang flag it should be OK merge

[PATCH] D116634: enable msan-eager-checks with -fsanitize-memory-param-retval.

2022-01-04 Thread Kevin Athey via Phabricator via cfe-commits
kda created this revision. kda added a reviewer: vitalybuka. Herald added subscribers: ormris, hiraditya. kda requested review of this revision. Herald added projects: clang, Sanitizers, LLVM. Herald added subscribers: llvm-commits, Sanitizers, cfe-commits. follow up to