asb added a comment. Thanks for this patch Guy. As just discussed in the RISC-V sync-up call, it would be helpful from a review perspective to write down at least a simple plain-text description of the changes to the psABI doc needed to reflect the BE ABI implemented by GCC (and soon LLVM), perhaps in an issue.
I think this patch is lacking some test coverage around things like fixup handling (e.g. the logic to swap fixups). 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. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128612/new/ https://reviews.llvm.org/D128612 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
