[PATCH] D31800: [llvm-pdbdump] Display padding bytes on record layout

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 10 11:53:24 PDT 2017


rnk added a comment.

Is it possible to write a test with an existing PDB in Inputs?



================
Comment at: llvm/tools/llvm-pdbdump/PrettyClassDefinitionDumper.cpp:145
+        ShowInLayoutMode = true;
+        LLVM_FALLTHROUGH;
+      default:
----------------
Why fall through to break? Seems less special to break.


https://reviews.llvm.org/D31800





More information about the llvm-commits mailing list