commit 46b3b059e9fbc1ff42b73edb81f9ad5b60386a75
author David Neto <dneto@google.com> Fri May 07 15:19:54 2021 +0000
committer Commit Bot service account <commit-bot@chromium.org> Fri May 07 15:19:54 2021 +0000
tree 72ec15fe4572aee242a320088b68aedb48f79185
parent fb13f025a34bb3926903779f5ae224b9f4385a38

Allow AST dumping even if resolution failed

Fixed: tint:674
Change-Id: I1a13a20c5c2c886d04e3d37a198109e62ad7dad2
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/50246
Auto-Submit: David Neto <dneto@google.com>
Commit-Queue: Ben Clayton <bclayton@google.com>
Kokoro: Kokoro <noreply+kokoro@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

samples/main.cc

diff --git a/samples/main.cc b/samples/main.cc
index 89567e3..5346d72 100644
--- a/samples/main.cc
+++ b/samples/main.cc
@@ -642,13 +642,14 @@
   if (program->Diagnostics().count() > 0) {
     diag_formatter.format(program->Diagnostics(), diag_printer.get());
   }
-  if (!program->IsValid()) {
-    return 1;
-  }
 
   if (options.dump_ast) {
     std::cout << std::endl << program->to_str(options.demangle) << std::endl;
   }
+
+  if (!program->IsValid()) {
+    return 1;
+  }
   if (options.parse_only) {
     return 1;
   }