[PATCH] D104639: [AIX][XCOFF] Add support for 64-bit file header and section header writing

Maryam via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 20 01:54:24 PDT 2021


MaryamBen added a comment.

When I did this work, I had no idea that I could remove "64-bit XCOFF object files are not supported yet." in order to create tests.
Unfortunately, I learned this late, 
Thank you so much for your clarifications.



================
Comment at: llvm/include/llvm/BinaryFormat/XCOFF.h:30
 constexpr size_t FileHeaderSize32 = 20;
+constexpr size_t FileHeaderSize64 = 24;
 constexpr size_t SectionHeaderSize32 = 40;
----------------
sfertile wrote:
> I'm not sure why these changes are showing up in this patch as they are all landed upstream already (they landed in the patch adding support for yaml2obj support for xcoff)
I think the patch you are talking about was recently merged. 
When I created the pacth (2 months ago) , these changes weren't applied.



================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:78
   const MCSectionXCOFF *const MCCsect;
-  uint32_t SymbolTableIndex;
-  uint32_t Address;
-  uint32_t Size;
+  uint64_t SymbolTableIndex;
+  uint64_t Address;
----------------
sfertile wrote:
> Isn't the 64-bit object format also limited to a 4 byte symbol table index?
Yes I made a mistake, I will change it right now.


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

https://reviews.llvm.org/D104639



More information about the llvm-commits mailing list