<p dir="ltr">Do I read it correctly as "LGTM but please use enum here and here?"</p>
<p dir="ltr">It's probably reasonable to put the enum in some coff related header shared between lib/AsmPrinter and lib/Tools</p>
<div class="gmail_quote">19 дек. 2013 г. 1:35 пользователь "Eric Christopher" <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  Not so fond of the formatting, but seems at least reasonable.<br>
<br>
<br>
================<br>
Comment at: tools/llvm-readobj/COFFDumper.cpp:669<br>
@@ +668,3 @@<br>
+    W.printHex("Magic", Magic);<br>
+    if (Magic != 4) {<br>
+      error(object_error::parse_failed);<br>
----------------<br>
4? An enum maybe or something else?<br>
<br>
================<br>
Comment at: tools/llvm-readobj/COFFDumper.cpp:694<br>
@@ +693,3 @@<br>
+      switch (SubSectionType) {<br>
+      case 0xF2: {<br>
+        // F2 is a PC to file:line table.  Some data to parse this subsection is<br>
----------------<br>
Same here with F2, F3, F4...<br>
<br>
================<br>
Comment at: tools/llvm-readobj/COFFDumper.cpp:787<br>
@@ +786,3 @@<br>
+          StringTable.data()[FilenameOffsetInF3 - 1] != '\0') {<br>
+        // Each string in an F3 subsection should be preceded by a null<br>
+        // character.<br>
----------------<br>
Preceded? Awesome.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2425" target="_blank">http://llvm-reviews.chandlerc.com/D2425</a><br>
</blockquote></div>