[AIX][XCOFF][NFC] Change the SectionLen field name of CSect Auxiliary entry to SectionOrLength.

[DiggerLin]

Summary

According the the XCOFF document,

If
Then
XTY_SD
x_scnlen contains the csect length.
XTY_LD
x_scnlen contains the symbol table index of the containing csect.
XTY_CM
x_scnlen contains the csect length.
XTY_ER
x_scnlen contains 0.

Change the SectionLen member name to SectionOrLength is more reasonable .

[hubert.reinterpretcast]

Thanks @DiggerLin. Just minor comments.

llvm/include/llvm/Object/XCOFFObjectFile.h

117

s/it contains//g;

s/XTY_SD,XTY_CM/XTY_SD or XTY_CM/;
s/is XTY_ER/is XTY_ER/;

[DiggerLin]

llvm/include/llvm/Object/XCOFFObjectFile.h

117

changed as suggestion.

[DiggerLin]

addressed comment

[hubert.reinterpretcast]

LGTM. Thanks again.

[• ham999]

In D68650#1700496, @hubert.reinterpretcast wrote:

LGTM. Thanks again.

[jasonliu]

Sorry I accidentally committed llvm/D68650.diff. I already removed it with another commit llvm-svn: 374181.

llvm/include/llvm/Object/XCOFFObjectFile.h

 };
 
 struct XCOFFStringTable {
   uint32_t Size;
   const char *Data;
 };
 
 struct XCOFFCsectAuxEnt32 {
-  support::ubig32_t SectionLen;
+  support::ubig32_t
+      SectionOrLength; // If the symbol type is XTY_SD or XTY_CM, the csect
+                       // length.
+                       // If the symbol type is XTY_LD, the symbol table
+                       // index of the containing csect. 
+                       // If the symbol type is XTY_ER, 0.
   support::ubig32_t ParameterHashIndex;
   support::ubig16_t TypeChkSectNum;
   uint8_t SymbolAlignmentAndType;
   XCOFF::StorageMappingClass StorageMappingClass;
   support::ubig32_t StabInfoIndex;
   support::ubig16_t StabSectNum;
 };
 

llvm/tools/llvm-readobj/XCOFFDumper.cpp

 
 void XCOFFDumper::printCsectAuxEnt32(const XCOFFCsectAuxEnt32 *AuxEntPtr) {
   assert(!Obj.is64Bit() && "32-bit interface called on 64-bit object file.");
 
   DictScope SymDs(W, "CSECT Auxiliary Entry");
   W.printNumber("Index",
                 Obj.getSymbolIndex(reinterpret_cast<uintptr_t>(AuxEntPtr)));
   if ((AuxEntPtr->SymbolAlignmentAndType & SymbolTypeMask) == XCOFF::XTY_LD)
-    W.printNumber("ContainingCsectSymbolIndex", AuxEntPtr->SectionLen);
+    W.printNumber("ContainingCsectSymbolIndex", AuxEntPtr->SectionOrLength);
   else
-    W.printNumber("SectionLen", AuxEntPtr->SectionLen);
+    W.printNumber("SectionLen", AuxEntPtr->SectionOrLength);
   W.printHex("ParameterHashIndex", AuxEntPtr->ParameterHashIndex);
   W.printHex("TypeChkSectNum", AuxEntPtr->TypeChkSectNum);
   // Print out symbol alignment and type.
   W.printNumber("SymbolAlignmentLog2",
                 (AuxEntPtr->SymbolAlignmentAndType & SymbolAlignmentMask) >>
                     SymbolAlignmentBitOffset);
   W.printEnum("SymbolType", AuxEntPtr->SymbolAlignmentAndType & SymbolTypeMask,
               makeArrayRef(CsectSymbolTypeClass));