[PATCH] D45972: [ELF][MIPS] Fix calculation of GP relative relocations in case of relocatable output

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 25 03:45:29 PDT 2018


grimar added a comment.

It seems you can produce .o files you have added (mips-micro-gp0-non-zero.o, mips-n64-gp0-non-zero.o) with use of llvm-mc, right?
Can you do that in this patch? I think we generally want to avoid binaries in inputs. You might be able to test produced input objects to check they contain
non-zero gp0 if you think that makes sense.

(Would also probably be nice to do that for mips-gp0-non-zero.o in a followup).



================
Comment at: test/ELF/mips-non-zero-gp0.s:26
+
+# Source code for mips-micro-gp0-non-zero.o:
+
----------------
The source code is missing.


================
Comment at: test/ELF/mips-non-zero-gp0.s:42
+
+# REQUIRES: mips
+
----------------
Please move it to the first line of the test.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D45972





More information about the llvm-commits mailing list