This is an archive of the discontinued LLVM Phabricator instance.

[RL78 patch 1] Recognize RL78 in triple parsing code
AcceptedPublic

Authored by SebastianPerta on May 16 2023, 10:13 AM.

Details

Summary

Hello all,

I'm sharing an initial set of patches that adds LLVM backend for RL78 architecture.
I'll prepare the patches incrementally, in a similar order to Xtensa.

Thank you!

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 16 2023, 10:13 AM
SebastianPerta requested review of this revision.May 16 2023, 10:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 16 2023, 10:13 AM
arsenm accepted this revision.May 16 2023, 10:45 AM

LGTM

llvm/CODE_OWNERS.TXT
266–269 ↗(On Diff #522682)

I'd split this to a patch that introduces some real functionality

This revision is now accepted and ready to land.May 16 2023, 10:45 AM

I'd split this to a patch that introduces some real functionality

Agreed, done. Thank you (this is what I saw in the Xtensa first patch this is why I did it).

arsenm accepted this revision.May 17 2023, 2:03 AM

Just a reminder than I don't have commit access. Can somebody commit the patch on my behalf if it's OK to do so. Thank you!