[PATCH] [lsan] [mips] adding support of lsan for mips/mipsel arch

Sergey Matveev earthdok at google.com
Tue Oct 7 08:34:13 PDT 2014


Thanks for working on this. We are unlikely to accept patches that enable LSan on 32-bit architectures. Experiments on x86 and ARM have shown that the smaller address space makes leak detection very unreliable, to the point where it's difficult to even test the tool. See e.g. the discussion at https://code.google.com/p/address-sanitizer/issues/detail?id=294

In-depth review of the 64-bit part of the patch will follow.

http://reviews.llvm.org/D5616






More information about the llvm-commits mailing list