[llvm-dev] Moving the AVR backend out of experimental (original) (raw)

Alex Bradbury via llvm-dev llvm-dev at lists.llvm.org
Tue Feb 18 04:22:18 PST 2020


On Tue, 18 Feb 2020 at 07:45, Dylan McKay via llvm-dev <llvm-dev at lists.llvm.org> wrote:

Should we just make it a normal target? My only remaining reservation here - the generic DebugInfo tests, which presumably due to an unimplemented 16-bit branch somewhere deep in the llvm-objdump callstack. The AVR backend passes virtually all of the LLVM test suite but these when avr-unknown-unknown is set as the default target. It feels like the inclusion of ~80 XFAILs for these DebugInfo tests would be the only wart on the experimental->official code review. I've had a couple long attempts debugging this one, but have not yet found the source. Fixing of the DebugInfo tests would make the AVR official backend pitch much more tenable.

A recent patch <https://reviews.llvm.org/D73911> indicated there were issues to be solved still with disassembly. Although I know we don't have hard and fast rules here, it feels to me like it would be worth resolving those issues prior to flipping the switch on the basis that a working disassembler is something most people would expect from a "supported" backend that implements the MC layer.

Best,

Alex



More information about the llvm-dev mailing list