labath wrote:

> Hi @labath, So the error was due to the endianness difference between Linux 
> and AIX. The previous failure with the new test case has gone now. I have 
> dropped a fix so that AIX binary is recognised on little endian platforms as 
> well, but if there is any better way to add this check, please do let me 
> know. Thanks!

Are XCOFF files always big endian? If so, then all you need is to add  a 
`DataExtractor::SetByteOrder()`. If not (and you want to support both 
endiannesses), then it would be better to add both cases to 
`XCOFFHeaderSizeFromMagic`.

https://github.com/llvm/llvm-project/pull/111814
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to