[PATCH] D33961: encode YAML scalars using double quotes so all values can be expressed

Fri Jun 9 14:55:50 PDT 2017

inglorion updated this revision to Diff 102077.
inglorion marked 3 inline comments as done.
inglorion added a comment.
Herald added a subscriber: hiraditya.

Implemented @zturner's suggestions

https://reviews.llvm.org/D33961

Files:
  clang-tools-extra/test/clang-tidy/fix.cpp
  clang-tools-extra/test/clang-tidy/google-module.cpp
  clang/unittests/Tooling/RefactoringTest.cpp
  clang/unittests/Tooling/ReplacementsYamlTest.cpp
  lld/test/COFF/pdb-none.test
  lld/test/COFF/pdb.test
  lld/test/ELF/lto/opt-remarks.ll
  lld/test/mach-o/bind-opcodes.yaml
  lld/test/mach-o/eh-frame-relocs-arm64.yaml
  lld/test/mach-o/objc-category-list-atom.yaml
  lld/test/mach-o/objc-image-info-pass-output.yaml
  lld/test/mach-o/parse-tlv-relocs-x86-64.yaml
  lld/test/mach-o/run-tlv-pass-x86-64.yaml
  lld/test/mach-o/sectcreate.yaml
  lld/test/mach-o/string-table.yaml
  llvm/lib/Support/YAMLTraits.cpp
  llvm/test/CodeGen/MIR/Generic/frame-info.mir
  llvm/test/CodeGen/MIR/X86/callee-saved-info.mir
  llvm/test/CodeGen/MIR/X86/constant-pool.mir
  llvm/test/CodeGen/MIR/X86/dynamic-regmask.ll
  llvm/test/CodeGen/MIR/X86/expected-named-register-in-allocation-hint.mir
  llvm/test/CodeGen/MIR/X86/frame-info-save-restore-points.mir
  llvm/test/CodeGen/MIR/X86/frame-info-stack-references.mir
  llvm/test/CodeGen/MIR/X86/function-liveins.mir
  llvm/test/CodeGen/MIR/X86/generic-instr-type.mir
  llvm/test/CodeGen/MIR/X86/jump-table-info.mir
  llvm/test/CodeGen/MIR/X86/register-operand-class.mir
  llvm/test/CodeGen/MIR/X86/roundtrip.mir
  llvm/test/CodeGen/MIR/X86/simple-register-allocation-hints.mir
  llvm/test/CodeGen/MIR/X86/spill-slot-fixed-stack-objects.mir
  llvm/test/CodeGen/MIR/X86/stack-object-debug-info.mir
  llvm/test/CodeGen/MIR/X86/stack-objects.mir
  llvm/test/CodeGen/MIR/X86/variable-sized-stack-objects.mir
  llvm/test/CodeGen/MIR/X86/virtual-registers.mir
  llvm/test/CodeGen/X86/GlobalISel/legalize-mul-scalar.mir
  llvm/test/CodeGen/X86/GlobalISel/legalize-mul-v128.mir
  llvm/test/CodeGen/X86/GlobalISel/legalize-mul-v256.mir
  llvm/test/CodeGen/X86/GlobalISel/legalize-mul-v512.mir
  llvm/test/CodeGen/X86/GlobalISel/regbankselect-AVX2.mir
  llvm/test/CodeGen/X86/GlobalISel/regbankselect-AVX512.mir
  llvm/test/CodeGen/X86/GlobalISel/regbankselect-X32.mir
  llvm/test/CodeGen/X86/GlobalISel/regbankselect-X86_64.mir
  llvm/test/CodeGen/X86/GlobalISel/select-add-v128.mir
  llvm/test/CodeGen/X86/GlobalISel/select-add-v256.mir
  llvm/test/CodeGen/X86/GlobalISel/select-add-v512.mir
  llvm/test/CodeGen/X86/GlobalISel/select-add-x32.mir
  llvm/test/CodeGen/X86/GlobalISel/select-add.mir
  llvm/test/CodeGen/X86/GlobalISel/select-cmp.mir
  llvm/test/CodeGen/X86/GlobalISel/select-constant.mir
  llvm/test/CodeGen/X86/GlobalISel/select-ext-x86-64.mir
  llvm/test/CodeGen/X86/GlobalISel/select-ext.mir
  llvm/test/CodeGen/X86/GlobalISel/select-gep.mir
  llvm/test/CodeGen/X86/GlobalISel/select-inc.mir
  llvm/test/CodeGen/X86/GlobalISel/select-leaf-constant.mir
  llvm/test/CodeGen/X86/GlobalISel/select-memop-scalar-x32.mir
  llvm/test/CodeGen/X86/GlobalISel/select-memop-scalar.mir
  llvm/test/CodeGen/X86/GlobalISel/select-memop-v128.mir
  llvm/test/CodeGen/X86/GlobalISel/select-memop-v256.mir
  llvm/test/CodeGen/X86/GlobalISel/select-memop-v512.mir
  llvm/test/CodeGen/X86/GlobalISel/select-mul-scalar.mir
  llvm/test/CodeGen/X86/GlobalISel/select-mul-vec.mir
  llvm/test/CodeGen/X86/GlobalISel/select-sub-v128.mir
  llvm/test/CodeGen/X86/GlobalISel/select-sub-v256.mir
  llvm/test/CodeGen/X86/GlobalISel/select-sub-v512.mir
  llvm/test/CodeGen/X86/GlobalISel/select-sub.mir
  llvm/test/CodeGen/X86/GlobalISel/select-trunc.mir
  llvm/test/CodeGen/X86/stack-protector-remarks.ll
  llvm/test/CodeGen/X86/virtual-registers-cleared-in-machine-functions-liveins.ll
  llvm/test/DebugInfo/PDB/Inputs/one-symbol.yaml
  llvm/test/DebugInfo/PDB/pdb-yaml-symbols.test
  llvm/test/DebugInfo/PDB/pdbdump-debug-subsections.test
  llvm/test/DebugInfo/PDB/pdbdump-source-names.test
  llvm/test/DebugInfo/PDB/pdbdump-yaml-types.test
  llvm/test/DebugInfo/PDB/pdbdump-yaml.test
  llvm/test/LTO/Resolution/X86/diagnostic-handler-remarks-with-hotness.ll
  llvm/test/LTO/Resolution/X86/diagnostic-handler-remarks.ll
  llvm/test/LTO/X86/diagnostic-handler-remarks-with-hotness.ll
  llvm/test/LTO/X86/diagnostic-handler-remarks.ll
  llvm/test/Object/obj2yaml-coff-section-aux-symbol.test
  llvm/test/Object/obj2yaml-invalid-reloc.test
  llvm/test/Object/obj2yaml.test
  llvm/test/ObjectYAML/MachO/DWARF-BigEndian.yaml
  llvm/test/ObjectYAML/MachO/DWARF-LittleEndian.yaml
  llvm/test/ObjectYAML/MachO/DWARF-debug_str.yaml
  llvm/test/ObjectYAML/MachO/dylib_dylinker_command.yaml
  llvm/test/ObjectYAML/MachO/export_trie.yaml
  llvm/test/ObjectYAML/MachO/null_string_entries.yaml
  llvm/test/ObjectYAML/MachO/out_of_order_linkedit.yaml
  llvm/test/ObjectYAML/MachO/symtab.yaml
  llvm/test/ObjectYAML/wasm/data_section.yaml
  llvm/test/ThinLTO/X86/diagnostic-handler-remarks-with-hotness.ll
  llvm/test/ThinLTO/X86/diagnostic-handler-remarks.ll
  llvm/test/Transforms/GVN/opt-remarks.ll
  llvm/test/Transforms/Inline/optimization-remarks-passed-yaml.ll
  llvm/test/Transforms/Inline/optimization-remarks-yaml.ll
  llvm/test/Transforms/LoopVectorize/X86/vectorization-remarks-missed.ll
  llvm/test/Transforms/WholeProgramDevirt/export-single-impl.ll
  llvm/test/Transforms/WholeProgramDevirt/export-uniform-ret-val.ll
  llvm/test/Transforms/WholeProgramDevirt/export-unique-ret-val.ll
  llvm/test/Transforms/WholeProgramDevirt/export-vcp.ll
  llvm/test/Transforms/WholeProgramDevirt/import-indir.ll
  llvm/test/tools/dsymutil/arch-option.test
  llvm/test/tools/dsymutil/debug-map-parsing.test
  llvm/test/tools/dsymutil/fat-binary-output.test
  llvm/test/tools/dsymutil/yaml-object-address-rewrite.test
  llvm/test/tools/llvm-xray/X86/convert-fdr-to-yaml.txt
  llvm/test/tools/llvm-xray/X86/convert-roundtrip.yaml
  llvm/test/tools/llvm-xray/X86/convert-to-yaml.txt
  llvm/test/tools/llvm-xray/X86/convert-with-standalone-instrmap.txt
  llvm/test/tools/llvm-xray/X86/convert-with-yaml-instrmap.txt
  llvm/unittests/ObjectYAML/YAMLTest.cpp
  llvm/unittests/Support/YAMLIOTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33961.102077.patch
Type: text/x-patch
Size: 265583 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170609/093dab16/attachment-0001.bin>