<br><br><div>On Wed Dec 18 2013 at 8:25:11 PM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p dir="ltr">Do I read it correctly as "LGTM but please use enum here and here?"</p></blockquote><div><br></div><div>Yep.</div><div>š</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<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></div></blockquote><div><br></div><div>Agreed. Probably in include/llvm/Support/COFF.h somewhere?</div><div><br></div><div>-eric</div><div>š</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>19 ÄÅË. 2013 Ç. 1:35 ÐÏÌØÚÏ×ÁÔÅÌØ "Eric Christopher" <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> ÎÁÐÉÓÁÌ:</div><div><br type="attribution"><blockquote 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></blockquote>