[Demangler] Remove no-op case of DEMANGLER_ASSERT

Author: kastiglione

lib/Demangling/DemanglerAssert.h

@@ -22,30 +22,25 @@
 #include "swift/Demangling/Demangle.h"
 #include "swift/Demangling/NamespaceMacros.h"
 
-#if SWIFT_RUNTIME
+#if !defined(NDEBUG) && !SWIFT_RUNTIME
 
-// In the runtime, DEMANGLER_ASSERT() returns an error
+// Except in the runtime, DEMANGLER_ASSERT() works like assert()
 #define DEMANGLER_ASSERT(expr, node)                                           \
   do {                                                                         \
     if (!(expr))                                                               \
-      return ManglingError(ManglingError::AssertionFailed, (node), __LINE__);  \
+      swift::Demangle::failAssert(__FILE__, __LINE__, node, #expr);            \
   } while (0)
 
-#elif !defined(NDEBUG)
+#else
 
-// If NDEBUG is not defined, DEMANGLER_ASSERT() works like assert()
+// In the runtime and non-asserts builds, DEMANGLER_ASSERT() returns an error
 #define DEMANGLER_ASSERT(expr, node)                                           \
   do {                                                                         \
     if (!(expr))                                                               \
-      swift::Demangle::failAssert(__FILE__, __LINE__, node, #expr);            \
+      return ManglingError(ManglingError::AssertionFailed, (node), __LINE__);  \
   } while (0)
 
-#else
-
-// Otherwise, DEMANGLER_ASSERT() does nothing
-#define DEMANGLER_ASSERT(expr, node)
-
-#endif // SWIFT_RUNTIME
+#endif // !defined(NDEBUG) && !SWIFT_RUNTIME
 
 namespace swift {
 namespace Demangle {