gbenyei added a comment.
In D128612#3617906 <https://reviews.llvm.org/D128612#3617906>, @jhenderson
wrote:
> Objcopy aspects look good, thanks.
Thanks
================
Comment at: llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp:554
.buildGraph();
- } else {
- assert((*ELFObj)->getArch() == Triple::riscv32 &&
- "Invalid triple for RISCV ELF object file");
+ } else if ((*ELFObj)->getArch() == Triple::riscv64be) {
+ auto &ELFObjFile = cast<object::ELFObjectFile<object::ELF64BE>>(**ELFObj);
----------------
jrtc27 wrote:
> Why switch to this order when before you've used 32, 64, 32be, 64be as the
> order
The order in the code before my changes is 64, 32. I guess for no good reason,
but I prefer not to re-order code while implementing a feature - it trashes git
history.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128612/new/
https://reviews.llvm.org/D128612
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
