This is an archive of the discontinued LLVM Phabricator instance.

gn build: Add build files for llvm/lib/Target/AVR
ClosedPublic

Authored by thakis on Sep 20 2019, 7:39 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

thakis created this revision.Sep 20 2019, 7:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 20 2019, 7:39 PM
mbonadei accepted this revision.Sep 21 2019, 5:26 AM

LGTM (only one comment about two possible missing deps).

llvm/utils/gn/secondary/llvm/lib/Target/AVR/Disassembler/BUILD.gn
15 ↗(On Diff #221150)

I think we also need to add //llvm/lib/Target/AVR/TargetInfo and //llvm/lib/Target/AVR:LLVMAVRCodeGen.

This revision is now accepted and ready to land.Sep 21 2019, 5:26 AM
thakis marked an inline comment as done.Sep 22 2019, 6:14 AM
thakis added inline comments.
llvm/utils/gn/secondary/llvm/lib/Target/AVR/Disassembler/BUILD.gn
15 ↗(On Diff #221150)

Good catch, thanks! Done. At least Sparc and probably several other targets need something like this too; I'll do changes to existing targets in a follow-up.

(And maybe, one day, working gn check…)

This revision was automatically updated to reflect the committed changes.