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

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

In https://reviews.llvm.org/D32991#776640, @xiangzhai wrote:

> Dear Rui,
>
> What about avr-objdump?
>
>   $ avr-objdump -v
>   GNU objdump (GNU Binutils) 2.27
>   Copyright (C) 2016 Free Software Foundation, Inc.
>   This program is free software; you may redistribute it under the terms of
>   the GNU General Public License version 3 or (at your option) any later version.
>   This program has absolutely no warranty.
>  
>   $ avr-objdump -d basic-avr
>  
>   basic-avr:     file format elf32-avr
>  
>  
>   Disassembly of section .text:
>  
>   00011000 <main>:
>      11000:       02 88           ldd     r0, Z+18        ; 0x12
>           ...
>  
>   00011004 <foo>:
>      11004:       02 88           ldd     r0, Z+18        ; 0x12
>           ...
>
>
> basic-avr was **generated** by:
>
>   $ llvm/build/bin/llvm-mc -filetype=obj -triple=avr-unknown-linux -mcpu=atmega328p llvm/tools/lld/test/ELF/basic-avr.s -o /tmp/basic-avr.o
>   $ llvm/build/bin/lld /tmp/basic-avr.o -o /tmp/basic-avr
>
>
> Regards,
>  Leslie Zhai


Yes, llvm-objdump (i.e. MC) misses the decoding for these two instructions. You may want to add that first.


Repository:
  rL LLVM

https://reviews.llvm.org/D32991