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

Leslie Zhai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 21:40:20 PDT 2017


xiangzhai added a comment.

Dear David,

Thanks for your hint!

> In the future, we should refactor LLVM so that it is something like this quick dirty patch F3351549: patch.diff <https://reviews.llvm.org/F3351549>
> 
> That way, we can use the same relocation code in LLD and LLVM.

@dylanmckay And I noticed that Dylan moved fixup_call from lib/Target/AVR/MCTargetDesc/AVRAsmBackend.cpp into include/llvm/Support/AVRRelocations.h so other LLVM's tools such as lld, llvm-objdump, etc are able to reuse the same relocaiton code, correct? then this patch is depends on Dylan's work, he is the owner of AVR target, but I could not confirm that I have enough time to work together with him owing to work assignment :(

Regards,
Leslie Zhai


Repository:
  rL LLVM

https://reviews.llvm.org/D32991





More information about the llvm-commits mailing list