[PATCH] D66969: Output XCOFF object text section

[Jason Liu via Phabricator via llvm-commits]

jasonliu added inline comments.


================
Comment at: llvm/lib/MC/MCXCOFFStreamer.cpp:78
+
+  // TODO: Handle Fixups Later
+
----------------
Minor nit: "Handle Fixups later."


================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:340
+    const Symbol &SymbolRef, const ControlSection &CSectionRef,
+    const int16_t Index, uint64_t SymbolOffset) {
+  // n_name, n_zeros, n_offset
----------------
top level const is not necessary for the Index parameter.
Should it be uint16_t?
Nit: Index -> SectionIndex 



================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:344
+  W.write<uint32_t>(CSectionRef.Address + SymbolOffset);
+  W.write<int16_t>(Index);
+  // Basic/Derived type. See the description of the n_type field for symbol
----------------
int16_t -> uint16_t?


================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:372
+void XCOFFObjectWriter::writeSymbolTableEntryForControlSection(
+    const ControlSection &CSectionRef, const int16_t Index,
+    XCOFF::StorageClass StorageClass) {
----------------
Index -> SectionIndex


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66969/new/

https://reviews.llvm.org/D66969