This is an archive of the discontinued LLVM Phabricator instance.

[AVR] Add calling convention parser tokens
ClosedPublic

Authored by dylanmckay on Jan 20 2016, 2:40 AM.

Details

Summary

Adds the 'avr_intrcc' and 'avr_signalcc' IR calling convention tokens to the parser.

Diff Detail

Event Timeline

dylanmckay updated this revision to Diff 45365.Jan 20 2016, 2:40 AM
dylanmckay retitled this revision from to [AVR] Add calling convention parser tokens.
dylanmckay updated this object.
dylanmckay added a reviewer: arsenm.
arsenm added inline comments.Jan 20 2016, 11:53 AM
test/CodeGen/AVR/calling-conv-interrupts.ll
2 ↗(On Diff #45365)

Why is this here?

test/CodeGen/AVR/calling-conv-signal.ll
2 ↗(On Diff #45365)

ditto

dylanmckay added inline comments.Jan 20 2016, 1:11 PM
test/CodeGen/AVR/calling-conv-interrupts.ll
2 ↗(On Diff #45365)

I added the AVR calling conventions in a previous commit, but forgot to add a test for them. I wrote one here, but marked it XFAIL because the target machine cannot be created yet.

Happy to put it in a separate commit if desired.

arsenm added inline comments.Jan 26 2016, 10:26 AM
test/CodeGen/AVR/calling-conv-interrupts.ll
2 ↗(On Diff #45365)

These codegen tests should be left for a backend commit where these work.

You should instead replace these with bitcode tests, e.g. see tests/Bitcode/*

dylanmckay updated this revision to Diff 46503.Jan 31 2016, 6:55 PM

Changed CodeGen tests to Bitcode tests

dylanmckay marked 4 inline comments as done.Jan 31 2016, 6:56 PM

Fixed up Matt's code review points.

arsenm accepted this revision.Mar 2 2016, 11:13 AM
arsenm edited edge metadata.

LGTM

This revision is now accepted and ready to land.Mar 2 2016, 11:13 AM
dylanmckay closed this revision.Mar 3 2016, 2:12 AM