[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