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

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 00:40:02 PDT 2017


ruiu added a comment.

They were more like questions than review comments. I assumed you know much more than I about AVR, and I wanted to know more about AVR to see where AVR support would eventually go. What is your eventual goal for AVR support?

As a patch, I think this is too small to submit. I want to something that is minimally useful (in some sense), such as one that can link a program that does nothing but is still valid. This patch is too small that we cannot test.

Why do you need getDynRel, isPicRel, and needsThunk?


Repository:
  rL LLVM

https://reviews.llvm.org/D32991





More information about the llvm-commits mailing list