[PATCH] D32991: [ELF] Initial migration of AVR target

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 19 11:06:16 PDT 2017


ruiu added a comment.

Please verify that the following program spins inside function `foo`.

  main:
    call foo
  
  foo:
    jmp foo

Please update basic-avr.s with this test.



================
Comment at: ELF/Target.cpp:2060
+  case R_AVR_32:
+  case R_AVR_CALL:
+    return R_ABS;
----------------
I asked you to return R_ABS for R_AVR_CALL, but I don't know about the AVR instruction set, so I may be wrong. Is R_AVR_CALL is actually an absolute relocation? It may be a relative one.


Repository:
  rL LLVM

https://reviews.llvm.org/D32991





More information about the llvm-commits mailing list