[PATCH] D141036: [JITLink] Add an initial implementation of JITLink for ELF/LoongArch

[Lang Hames via Phabricator via llvm-commits]

lhames accepted this revision.
lhames added a comment.
This revision is now accepted and ready to land.

I can't speak to the architecture specific details as I don't know Loongarch, but if the tests are passing this looks excellent to me -- The layout and style fits very well with the existing backends.

Do you have commit access?



================
Comment at: llvm/lib/ExecutionEngine/JITLink/ELF.cpp:93-94
     return;
+  case Triple::loongarch32:
+  case Triple::loongarch64:
+    link_ELF_loongarch(std::move(G), std::move(Ctx));
----------------
Looks like the new backend will handle both `loongarch32` and `loongarch64`. Are there any differences at the linker level?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141036