[PATCH] D115103: Leak Sanitizer port to Windows

Vitaly Buka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 22 16:26:44 PST 2021


vitalybuka added a comment.

In D115103#3207210 <https://reviews.llvm.org/D115103#3207210>, @clemenswasser wrote:

> The problem is, that `LSAN_BASE` is not always `use_stacks=0:use_registers=0`.

Off-cause , I am asking to replace with correct value each test.

> Also there are many test cases, where `LSAN_BASE` gets used multiple times.
> Removing it would result in bad repetition.

I don't like LSAN_BASE even without Windows.
Usually when test fail, I have a single command line to reproduce or put into debugger for just the step. Here I need to cherry-pick two line: LSAN_BASE=  and actual step
also in most case it used just twice, so it's not a big loss in repetition


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

https://reviews.llvm.org/D115103



More information about the cfe-commits mailing list