Add BYN_ENABLE_ASSERTSION option to allow assertions to be disabled. (#2500)

We always enable assertions by default, but this options allows for
a build without them.

Fix all errors in the ASSERTIONS=OFF build, even though we don't
normally build this its good to keep it building.

1 files changed, 2 insertions, 0 deletions

diff --git a/src/passes/PostAssemblyScript.cpp b/src/passes/PostAssemblyScript.cpp
index eb2b1bfb4..4e41c8f08 100644
--- a/src/passes/PostAssemblyScript.cpp
+++ b/src/passes/PostAssemblyScript.cpp
@@ -101,6 +101,7 @@ static bool isRetain(LocalSet* expr) {
   return false;
 }
 
+#ifndef NDEBUG
 // Tests if the given location is that of a full retain pattern.
 static bool isRetainLocation(Expression** expr) {
   if (expr != nullptr) {
@@ -110,6 +111,7 @@ static bool isRetainLocation(Expression** expr) {
   }
   return false;
 }
+#endif
 
 // Tests if the given call calls release. Note that this differs from what we
 // consider a full release pattern, which must also get a local.