[PATCH] D22579: [pdbdump] Use the "flow" style to print out a sequence of uint32_t.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 12:00:53 PDT 2016


ruiu created this revision.
ruiu added a reviewer: zturner.
ruiu added a subscriber: llvm-commits.

Lists can be written either with "-" or "[]" in YAML.

https://reviews.llvm.org/D22579

Files:
  test/DebugInfo/PDB/pdbdump-yaml.test
  tools/llvm-pdbdump/PdbYaml.h

Index: tools/llvm-pdbdump/PdbYaml.h
===================================================================
--- tools/llvm-pdbdump/PdbYaml.h
+++ tools/llvm-pdbdump/PdbYaml.h
@@ -105,7 +105,7 @@
 }
 }
 
-LLVM_YAML_IS_SEQUENCE_VECTOR(uint32_t)
+LLVM_YAML_IS_FLOW_SEQUENCE_VECTOR(uint32_t)
 LLVM_YAML_IS_SEQUENCE_VECTOR(llvm::pdb::yaml::NamedStreamMapping)
 LLVM_YAML_IS_SEQUENCE_VECTOR(llvm::pdb::yaml::StreamBlockList)
 
Index: test/DebugInfo/PDB/pdbdump-yaml.test
===================================================================
--- test/DebugInfo/PDB/pdbdump-yaml.test
+++ test/DebugInfo/PDB/pdbdump-yaml.test
@@ -13,62 +13,29 @@
 ; YAML-NEXT:     Unknown1:        0
 ; YAML-NEXT:     BlockMapAddr:    24
 ; YAML-NEXT:   NumDirectoryBlocks: 1
-; YAML-NEXT:   DirectoryBlocks:
-; YAML-NEXT:     - 23
+; YAML-NEXT:   DirectoryBlocks: [ 23 ]
 ; YAML-NEXT:   NumStreams:      17
 ; YAML-NEXT:   FileSize:        102400
-; YAML-NEXT: StreamSizes:
-; YAML-NEXT:   - 40
-; YAML-NEXT:   - 118
-; YAML-NEXT:   - 5392
-; YAML-NEXT:   - 739
-; YAML-NEXT:   - 784
-; YAML-NEXT:   - 0
-; YAML-NEXT:   - 556
-; YAML-NEXT:   - 604
-; YAML-NEXT:   - 104
-; YAML-NEXT:   - 0
-; YAML-NEXT:   - 160
-; YAML-NEXT:   - 32
-; YAML-NEXT:   - 308
-; YAML-NEXT:   - 239
-; YAML-NEXT:   - 520
-; YAML-NEXT:   - 308
-; YAML-NEXT:   - 68
+; YAML-NEXT: StreamSizes: [ 40, 118, 5392, 739, 784, 0, 556, 604, 104, 0, 160, 32,
+; YAML-NEXT:                308, 239, 520, 308, 68 ]
 ; YAML-NEXT: StreamMap:
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 8
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 19
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 18
-; YAML-NEXT:       - 17
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 14
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 20
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 11
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 13
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 12
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 10
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 15
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 6
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 16
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 7
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 21
-; YAML-NEXT:   - Stream:
-; YAML-NEXT:       - 22
+; YAML-NEXT:   - Stream: [ 8 ]
+; YAML-NEXT:   - Stream: [ 19 ]
+; YAML-NEXT:   - Stream: [ 18, 17 ]
+; YAML-NEXT:   - Stream: [ 14 ]
+; YAML-NEXT:   - Stream: [ 20 ]
+; YAML-NEXT:   - Stream: [ ]
+; YAML-NEXT:   - Stream: [ 11 ]
+; YAML-NEXT:   - Stream: [ 13 ]
+; YAML-NEXT:   - Stream: [ 12 ]
+; YAML-NEXT:   - Stream: [ ]
+; YAML-NEXT:   - Stream: [ 10 ]
+; YAML-NEXT:   - Stream: [ 15 ]
+; YAML-NEXT:   - Stream: [ 6 ]
+; YAML-NEXT:   - Stream: [ 16 ]
+; YAML-NEXT:   - Stream: [ 7 ]
+; YAML-NEXT:   - Stream: [ 21 ]
+; YAML-NEXT:   - Stream: [ 22 ]
 ; YAML-NEXT: PdbStream:
 ; YAML-NEXT:   Age:             1
 ; YAML-NEXT:   Guid:            '{0B355641-86A0-A249-896F-9988FAE52FF0}'


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22579.64723.patch
Type: text/x-patch
Size: 3002 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160720/20c75c87/attachment.bin>


More information about the llvm-commits mailing list