search

avr-llvm / llvm

[MERGED UPSTREAM] AVR backend for the LLVM compiler library
220 stars 21 forks source link

Place architecture value inside ELF header #157

Closed dylanmckay closed 9 years ago

dylanmckay commented 9 years ago

This fixes #155.

@agnat Can you look over this? It touches a lot of code.

agnat commented 9 years ago

Will do.

agnat commented 9 years ago

I finished my review of the functional change. LGTM. That leaves us with the tables. Could you paste links to your sources? Then I'll give 'em a proof read, too.

dylanmckay commented 9 years ago

The MCU, family, and architecture information is taken from tc-avr.c from binutils.

Good luck!

dylanmckay commented 9 years ago

@agnat Thanks for the comments, I've pushed the changes :)

dylanmckay commented 9 years ago

I'm gonna merge this now so it can be reviewed later. I am very certain it doesn't contain any regressions (I double checked the tables as well).