[tblgen] Disable Leak detection for ASan/GCC and LSan/LLVM

[krytarowski]

Summary

Add support for sanitizing TableGen.cpp with ASan/GCC and LSan/LLVM.

[krytarowski]

Once GCC will grow support for __SANITIZE_LEAK__ (already pending in review) there will be included as a separate revision support for LSan/GCC.

CC: @jakubjelinek

[fjricci]

Interesting - I had tried to add has_feature for lsan to clang in the past and couldn’t get the change accepted. Nice to see it works now :)

[krytarowski]

In D67908#1678816, @fjricci wrote:

Interesting - I had tried to add has_feature for lsan to clang in the past and couldn’t get the change accepted. Nice to see it works now :)

We use it now in the NetBSD public headers [1] and keep __SANITIZE_LEAK__ as our local GCC extension... hopefully the GCC upstream will merge it as diverging from LLVM and from GCC downstream users.. is a cost.

[1] https://github.com/NetBSD/src/commit/e08b61780bb6fece3ba3f58a3dc6991ee831c41e

[fjricci]

Ah okay, that makes sense

llvm/trunk/utils/TableGen/TableGen.cpp

int main(int argc, char **argv) {
   PrettyStackTraceProgram X(argc, argv);
   cl::ParseCommandLineOptions(argc, argv);
 
   llvm_shutdown_obj Y;
 
   return TableGenMain(argv[0], &LLVMTableGenMain);
 }
 
-#ifdef __has_feature
-#if __has_feature(address_sanitizer)
+#ifndef __has_feature
+#define __has_feature(x) 0
+#endif
+
+#if __has_feature(address_sanitizer) || defined(__SANITIZE_ADDRESS__) ||       \
+    __has_feature(leak_sanitizer)
+
 #include <sanitizer/lsan_interface.h>
 // Disable LeakSanitizer for this binary as it has too many leaks that are not
 // very interesting to fix. See compiler-rt/include/sanitizer/lsan_interface.h .
 LLVM_ATTRIBUTE_USED int __lsan_is_turned_off() { return 1; }
-#endif  // __has_feature(address_sanitizer)
-#endif  // defined(__has_feature)
+
+#endif