hubert.reinterpretcast added a comment.

In D96033#2764946 <https://reviews.llvm.org/D96033#2764946>, 
@hubert.reinterpretcast wrote:

>> If that works on your platform I will happily open a review for the changes.
>
> From the behaviour observed in the 64-bit build, the 32-bit case might not 
> fail similarly. I need to see if I can get a 32-bit build going. For now, the 
> patch would be welcome (without changing from `UNSUPPORTED` to `XFAIL`).

The 32-bit case fails elsewhere:

  clang-repl> clang-repl> clang-repl> clang-repl> clang-repl> clang-repl> 
clang-repl> Not yet implemented!
  UNREACHABLE executed at 
/home/hstong/.Nrtphome/.Liodine/llcrossbld/dev/llvm-project/llvm/lib/Object/XCOFFObjectFile.cpp:219!
  IOT/Abort trap (core dumped)

So, I don't think the 32-bit and 64-bit cases are going to be synchronized.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96033

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to