[Lldb-commits] [PATCH] D128612: RISC-V big-endian support implementation

Wed May 17 03:36:12 PDT 2023

asb added a comment.

In D128612#4349259 <https://reviews.llvm.org/D128612#4349259>, @djtodoro wrote:

> In D128612#4345912 <https://reviews.llvm.org/D128612#4345912>, @asb wrote:
>
>> In D128612#4337037 <https://reviews.llvm.org/D128612#4337037>, @djtodoro wrote:
>>
>>> Hi! I am wondering if someone knows what is the status of this.
>>
>> I've not seen any further progress. I think it needs at a minimum a PR against the psABI doc for big endian that we can review against.
>
> OK. Can you please share with me the GCC PR, so I can take a look how it should be done :)

Based on comments here it sounds like GCC merged some level of support without submitting corresponding changes to the psABI. I don't have those patches to hand, but you can probably find them on the GCC patches list. I think we've had enough unexpected compatibility issues by now that it's not really justifiable to merge something that's meant to be a standard ABI without having the corresponding psABI patch (even if it appears simple). I don't think anyone has posted such a psABI patch but there is/was a tracking issue for it on the psABI repo.

@kito-cheng is the bigendian RISC-V work in GCC still active at all? I'm wondering if there's a connection you could help make for Djordje.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128612