[PATCH] D30664: [DWARFv5] Update definitions to match published spec

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 15:01:07 PST 2017


probinson added a comment.

In https://reviews.llvm.org/D30664#693714, @clayborg wrote:

> Seem like a lot of the Dwarf.def macros could use an extra "version" parameter. Might help code to verify that certain tags, attributes, forms, or more are valid for the current DWARF version.


I am definitely moving in the direction of version checking for forms.  Version checking is not so important for other things, as in general anything the consumer doesn't understand can be skipped without having the world explode.  But if a consumer doesn't understand a form, the entire section is toast.


Repository:
  rL LLVM

https://reviews.llvm.org/D30664





More information about the llvm-commits mailing list