[PATCH] D104644: [AIX][XCOFF] Support 64-bit relocation writing and related tests

Fri Jun 25 07:02:16 PDT 2021

MaryamBen added a comment.

In D104644#2837937 <https://reviews.llvm.org/D104644#2837937>, @jhenderson wrote:

> In D104644#2836029 <https://reviews.llvm.org/D104644#2836029>, @MaryamBen wrote:
>
>> In D104644#2832538 <https://reviews.llvm.org/D104644#2832538>, @jhenderson wrote:
>>
>>> This patch from the description is about 64-bit relocation support, so I'm not sure why there's so much changing in the testing. You shouldn't need so many tests for ensuring you have 64-bit relocation support, which implies that the existing tests aren't well designed - they are testing things in a too catch-all manner, and should probably be rewritten to test a small subset of functionality each. Furthermore, why does every test need to cover 64-bit behaviour?
>>
>> Since I had the fatal error report in 64-bit, I couldn't check the file/section header and symbol table tests. Once I removed this part of the code, I had to implement tests even for the other parts.
>>
>>   report_fatal_error("64-bit XCOFF object files are not supported yet.");
>>
>> So maybe I could just change the description or do you have another suggestion ?
>
> Could most of the test be rewritten to not require relocations? It seems like relocations shouldn't be necessary in many situations (assuming they work similarly to how they do in ELF). That would allow those tests to be updated as you make the other patches in the test series, which in turn provides accompanying testing for them.

>From what I know there is no way to ensure testing without relocations, even for a basic code.
An XCOFF/AIX person might confirm !

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

https://reviews.llvm.org/D104644