From 1abfe595aae2baa68c2888e53ac7f2890f9a75b5 Mon Sep 17 00:00:00 2001
From: Sam Clegg <sbc@chromium.org>
Date: Tue, 15 Oct 2019 14:41:55 -0700
Subject: Use early return in wasm-opt.cpp. NFC (#2387)

---
 src/tools/wasm-opt.cpp | 45 +++++++++++++++++++++++----------------------
 1 file changed, 23 insertions(+), 22 deletions(-)

(limited to 'src/tools/wasm-opt.cpp')

diff --git a/src/tools/wasm-opt.cpp b/src/tools/wasm-opt.cpp
index 130e284c2..33e9a0396 100644
--- a/src/tools/wasm-opt.cpp
+++ b/src/tools/wasm-opt.cpp
@@ -363,29 +363,30 @@ int main(int argc, const char* argv[]) {
   }
 
   if (options.extra.count("output") == 0) {
-    std::cerr << "(no output file specified, not emitting output)\n";
-  } else {
-    if (options.debug) {
-      std::cerr << "writing..." << std::endl;
-    }
-    ModuleWriter writer;
-    writer.setDebug(options.debug);
-    writer.setBinary(emitBinary);
-    writer.setDebugInfo(options.passOptions.debugInfo);
-    if (outputSourceMapFilename.size()) {
-      writer.setSourceMapFilename(outputSourceMapFilename);
-      writer.setSourceMapUrl(outputSourceMapUrl);
-    }
-    writer.write(*curr, options.extra["output"]);
+    std::cerr << "no output file specified, not emitting output\n";
+    return 0;
+  }
 
-    if (extraFuzzCommand.size() > 0) {
-      auto secondOutput = runCommand(extraFuzzCommand);
-      std::cout << "[extra-fuzz-command second output:]\n"
-                << firstOutput << '\n';
-      if (firstOutput != secondOutput) {
-        std::cerr << "extra fuzz command output differs\n";
-        abort();
-      }
+  if (options.debug) {
+    std::cerr << "writing..." << std::endl;
+  }
+  ModuleWriter writer;
+  writer.setDebug(options.debug);
+  writer.setBinary(emitBinary);
+  writer.setDebugInfo(options.passOptions.debugInfo);
+  if (outputSourceMapFilename.size()) {
+    writer.setSourceMapFilename(outputSourceMapFilename);
+    writer.setSourceMapUrl(outputSourceMapUrl);
+  }
+  writer.write(*curr, options.extra["output"]);
+
+  if (extraFuzzCommand.size() > 0) {
+    auto secondOutput = runCommand(extraFuzzCommand);
+    std::cout << "[extra-fuzz-command second output:]\n" << firstOutput << '\n';
+    if (firstOutput != secondOutput) {
+      std::cerr << "extra fuzz command output differs\n";
+      abort();
     }
   }
+  return 0;
 }
-- 
cgit v1.2.3