This patch adds test for MIPS64R6 relocations, corrects check expressions for R_MIPS_26 and R_MIPS_PC16 relocations in MIPS64R2 test, and adds run for big endian in MIPS64R2 test.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM
From the commit message:
changes some check expressions in MIPS64R2 test
Could you say 'correct' instead of change and summarize the correction?
lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp | ||
---|---|---|
684 | This isn't for this patch just something I thought of while reading: Should we warn if the 'x & ~0x7' masks any bits? I think we should. Likewise for the 'x & 0x3' cases. |
lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp | ||
---|---|---|
684 | Yes, we should warn. |
Comment Actions
Rebased test/ExecutionEngine/RuntimeDyld/Mips/ELF_Mips64r2N64_PIC_relocations.s to current trunk.
This isn't for this patch just something I thought of while reading:
Should we warn if the 'x & ~0x7' masks any bits? I think we should. Likewise for the 'x & 0x3' cases.