[PATCH] D155383: [clang][AST] TextNodeDumper learned to output exception specifications

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

In D155383#4507928 <https://reviews.llvm.org/D155383#4507928>, @strimo378 wrote:

> There are many ast-dump tests about decls but there are nearly no other type tests.

We should still be adding tests for these changes, to ensure we don't regress the behavior later by accident. It also helps the code reviewers to see what the output looks like and whether we spot any concerns with it.



================
Comment at: clang/lib/AST/TextNodeDumper.cpp:1549
+  case EST_DynamicNone:
+    OS << " exceptionspec_dynamicnone";
+    break;
----------------



================
Comment at: clang/lib/AST/TextNodeDumper.cpp:1555
+  case EST_MSAny:
+    OS << " exceptionspec_msany";
+    break;
----------------



================
Comment at: clang/lib/AST/TextNodeDumper.cpp:1561
+  case EST_BasicNoexcept:
+    OS << " exceptionspec_basicnoexcept";
+    break;
----------------



================
Comment at: clang/lib/AST/TextNodeDumper.cpp:1564
+  case EST_DependentNoexcept:
+    OS << " exceptionspec_dependentnoexcept";
+    break;
----------------



================
Comment at: clang/lib/AST/TextNodeDumper.cpp:1567
+  case EST_NoexceptFalse:
+    OS << " exceptionspec_noexceptfalse";
+    break;
----------------



================
Comment at: clang/lib/AST/TextNodeDumper.cpp:1570
+  case EST_NoexceptTrue:
+    OS << " exceptionspec_noexcepttrue";
+    break;
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155383