This is an archive of the discontinued LLVM Phabricator instance.

Add 'shave' processor name to Triple
ClosedPublic

Authored by dougk on Jun 12 2015, 11:05 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

dougk updated this revision to Diff 27585.Jun 12 2015, 11:05 AM
dougk retitled this revision from to Add 'shave' processor name to Triple.
dougk updated this object.
dougk edited the test plan for this revision. (Show Details)
dougk added a subscriber: Unknown Object (MLST).
chandlerc accepted this revision.Jun 12 2015, 11:13 AM
chandlerc added a reviewer: chandlerc.
chandlerc added a subscriber: chandlerc.

LGTM with minor tweak to wording.

You should also mention in the commit log when you submit this that the purpose initially is just to allow Clang to process these triples and redirect compilation to the Movidius dev kit.

include/llvm/ADT/Triple.h
87 ↗(On Diff #27585)

I don't think this is really the SoC.

I would say: "SHAVE: Movidius Myriad 2 vector VLIW processors" as that's how http://www.movidius.com/our-technology/myriad-2-platform/ describes them.

This revision is now accepted and ready to land.Jun 12 2015, 11:13 AM
This revision was automatically updated to reflect the committed changes.