r198885 - Revert "Disable LeakSanitizer in TableGen binaries, see PR18325"

Richard Smith richard at metafoo.co.uk
Thu Jan 9 14:07:59 PST 2014


Please back out this revert. We are far from achieving consensus on this
revert being appropriate.


On Thu, Jan 9, 2014 at 11:43 AM, Alp Toker <alp at nuanti.com> wrote:

> Author: alp
> Date: Thu Jan  9 13:43:17 2014
> New Revision: 198885
>
> URL: http://llvm.org/viewvc/llvm-project?rev=198885&view=rev
> Log:
> Revert "Disable LeakSanitizer in TableGen binaries, see PR18325"
>
> To declare or define reserved identifers is undefined behaviour in standard
> C++. This needs to be addressed in compiler-rt before it can be used in
> LLVM.
>
> See the list discussion for details.
>
> This reverts commit r198858.
>
> Modified:
>     cfe/trunk/utils/TableGen/TableGen.cpp
>
> Modified: cfe/trunk/utils/TableGen/TableGen.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/utils/TableGen/TableGen.cpp?rev=198885&r1=198884&r2=198885&view=diff
>
> ==============================================================================
> --- cfe/trunk/utils/TableGen/TableGen.cpp (original)
> +++ cfe/trunk/utils/TableGen/TableGen.cpp Thu Jan  9 13:43:17 2014
> @@ -263,10 +263,3 @@ int main(int argc, char **argv) {
>
>    return TableGenMain(argv[0], &ClangTableGenMain);
>  }
> -
> -extern "C" {
> -// Disable LeakSanitizer for this binary as it has too many leaks that
> are not
> -// very interesting to fix. __lsan_is_turned_off is explained in
> -// compiler-rt/include/sanitizer/lsan_interface.h
> -int __lsan_is_turned_off() { return 1; }
> -}  // extern "C"
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140109/1cc44da8/attachment.html>


More information about the cfe-commits mailing list