[PATCH] D58559: emit '(assertions enabled)' in the version string for a build of clang with assertions enabled

[Alex Lorenz via Phabricator via cfe-commits]

arphaman updated this revision to Diff 188023.
arphaman marked an inline comment as done.
arphaman added a comment.

Use `"(+asserts)"` as suggested.


Repository:
  rC Clang

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

https://reviews.llvm.org/D58559

Files:
  lib/Basic/Version.cpp


Index: lib/Basic/Version.cpp
===================================================================
--- lib/Basic/Version.cpp
+++ lib/Basic/Version.cpp
@@ -111,6 +111,11 @@
       OS << LLVMRepo << ' ';
     OS << LLVMRev << ')';
   }
+
+  // Differentiate between the version with and without assertions.
+#ifndef NDEBUG
+  OS << " (+asserts)";
+#endif
   return OS.str();
 }
 



Index: lib/Basic/Version.cpp
===================================================================
--- lib/Basic/Version.cpp
+++ lib/Basic/Version.cpp
@@ -111,6 +111,11 @@
       OS << LLVMRepo << ' ';
     OS << LLVMRev << ')';
   }
+
+  // Differentiate between the version with and without assertions.
+#ifndef NDEBUG
+  OS << " (+asserts)";
+#endif
   return OS.str();
 }
 

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits