<div dir="ltr">*nod* Agreed - I think that's worth considering (& then there's no issue with needing to put the vendor in the same namespace/enum/grouping as the version, too)<br><br>Though probably vendor extensions won't be versioned the same way the standard is anyway (eg: they might be added part-way through a standard, so some versions of the vendors tools that can handle DWARF4 can handle this vendor extension, and some can't).. so I'm not sure if that's all to helpful. Maybe it is... </div><br><div class="gmail_quote"><div dir="ltr">On Fri, Mar 10, 2017 at 4:28 PM Greg Clayton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I wonder since we are adding new macro parameters to some macros if we shouldn't add two params: version and vendor for anything that uses the special versions. Currently you are overloading version with vendor into a single integer, why not add both version and vendor? Then vendor could be<br class="gmail_msg">
<br class="gmail_msg">
  DWARF_VENDOR_DWARF = 0, // DWARF as ratified by the latest DWARF standard<br class="gmail_msg">
  DWARF_VENDOR_APPLE = 1,<br class="gmail_msg">
  DWARF_VENDOR_BORDLAND = 2,<br class="gmail_msg">
  ...<br class="gmail_msg">
<br class="gmail_msg">
This would still allow vendors to specify a version value for their user enums.<br class="gmail_msg">
<br class="gmail_msg">
Other than that it looks nice. It will really help with "llvm-dwarfdump --verify" in the future.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D30785" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D30785</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>