<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 17, 2014 at 5:23 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank" class="cremed">chandlerc@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
================<br>
Comment at: include/llvm/IR/Module.h:606<br>
@@ +605,3 @@<br>
+<br>
+  /// Return Dwarf Version by checking module flags.<br>
+  unsigned getDwarfVersion() const;<br>
----------------<br>
Go ahead and put a '\brief' on this.<br></blockquote><div><br></div><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
================<br>
Comment at: include/llvm/IR/Module.h:607<br>
@@ +606,3 @@<br>
+  /// Return Dwarf Version by checking module flags.<br>
+  unsigned getDwarfVersion() const;<br>
+<br>
----------------<br>
Any reason not to return the strongly typed enum?<br></blockquote><div><br></div><div>I copied and pasted from the original implementation. From what I can tell, the version number is an unsigned value. Dwarf.h is playing fast and loose with this. The only strong enum is for the value '4'. Everything else is just an integer.</div>
</div></div></div>