[PATCH] D84618: [DWARFYAML] Add support for emitting custom range list content.

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 01:05:09 PDT 2020


labath added a comment.

Unsurprisingly, I like this. :) It's simple, and can be used by obj2yaml (I hope) to represent unparsable range lists.



================
Comment at: llvm/lib/ObjectYAML/DWARFYAML.cpp:250-251
+  IO.mapOptional("Content", ListEntries.Content);
+  if (ListEntries.Entries && ListEntries.Content)
+    IO.setError("Entries and Content can't be used together");
 }
----------------
I'm not sure if there's anything wrong with this, but the docs seem to [[ https://llvm.org/docs/YamlIO.html#validation | recommend ]] using a `validate` method to do this kind of thing...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84618





More information about the llvm-commits mailing list