[llvm] c296c35 - [dsymutil] Re-enable 'auto' verification as the default

Jonas Devlieghere via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 16:25:52 PDT 2023


Author: Jonas Devlieghere
Date: 2023-08-17T16:25:47-07:00
New Revision: c296c35e9e2f9338d68a43865bc5e5cfa71b58cf

URL: https://github.com/llvm/llvm-project/commit/c296c35e9e2f9338d68a43865bc5e5cfa71b58cf
DIFF: https://github.com/llvm/llvm-project/commit/c296c35e9e2f9338d68a43865bc5e5cfa71b58cf.diff

LOG: [dsymutil] Re-enable 'auto' verification as the default

This patch re-enabled auto as the default verification mode when
building with assertions or expensive checks. This was disabled in
c2d2f724fe70 because of an issue with temp files on Windows, which was
fixed in 519490f26260.

Added: 
    

Modified: 
    llvm/tools/dsymutil/dsymutil.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/tools/dsymutil/dsymutil.cpp b/llvm/tools/dsymutil/dsymutil.cpp
index 1dffe0edc4dff9..69e6ee19fb9689 100644
--- a/llvm/tools/dsymutil/dsymutil.cpp
+++ b/llvm/tools/dsymutil/dsymutil.cpp
@@ -92,7 +92,7 @@ enum class DWARFVerify : uint8_t {
   All = Input | Output,
   Auto = Input | OutputOnValidInput,
 #if !defined(NDEBUG) || defined(EXPENSIVE_CHECKS)
-  Default = None // FIXME: Auto
+  Default = Auto
 #else
   Default = None
 #endif


        


More information about the llvm-commits mailing list