[PATCH] D47126: [RISCV] Add missing fixup_riscv_call in MCFixupKindInfo table

[Alex Bradbury via Phabricator via llvm-commits]

asb added inline comments.


================
Comment at: lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.cpp:280
+  case RISCV::fixup_riscv_call: {
+    uint64_t UpperImm = (Value + 0x800ULL) & 0xfffff000ULL;
+    uint64_t LowerImm = Value & 0xfffULL;
----------------
It would be helpful to have a comment to explain the addition of 0x800ULL.


================
Comment at: test/CodeGen/RISCV/function-call.ll:1-3
+; RUN: llc -mtriple=riscv32 -filetype=obj < %s\
+; RUN: | llvm-objdump -triple=riscv32 -mattr=-relax -d -riscv-no-aliases -\
+; RUN: | FileCheck -check-prefix=CHECK %s
----------------
Now this test has changed to just checking the fixup is resolved when -mattr=-relax I think it makes much more sense as an MC layer test. Perhaps expanding test/MC/RISCV/fixups.s? We need some reassurance that the fixup calculation is correct, even for offsets > 12 bits.

It looks like you have to use .space as in fixups.s, as the following still results in a relocation, even though I would have expected it not to:
```
.equ addr, 0xdeadbeef
call addr
```


Repository:
  rL LLVM

https://reviews.llvm.org/D47126