[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