This is an archive of the discontinued LLVM Phabricator instance.

Add info on running remote test suite to the website
ClosedPublic

Authored by labath on Apr 7 2015, 9:35 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

labath updated this revision to Diff 23345.Apr 7 2015, 9:35 AM
labath retitled this revision from to Add info on running remote test suite to the website.
labath updated this object.
labath edited the test plan for this revision. (Show Details)
labath added reviewers: tberghammer, zturner.
labath added a subscriber: Unknown Object (MLST).
zturner edited edge metadata.Apr 7 2015, 9:37 AM

I know this is more work, but would it be possible to split all the testing
stuff out into a totally separate page on the sidebar?

labath updated this revision to Diff 23406.Apr 8 2015, 1:53 AM
labath edited edge metadata.

That sounds like a good idea. I have created a separate test page. Please take another look.

tberghammer edited edge metadata.Apr 8 2015, 4:07 AM

Looks good with 2 minor comment

www/test.html
72–73 ↗(On Diff #23406)

It can be also specified by '-t' or by '--threads'

86–88 ↗(On Diff #23406)

Probably add a note that running the test suit against a remote target with multiple threads isn't fully supported yet.

labath updated this revision to Diff 23471.Apr 9 2015, 3:52 AM
labath edited edge metadata.

Address review comments.

tberghammer accepted this revision.Apr 9 2015, 6:30 AM
tberghammer edited edge metadata.

LGTM

This revision is now accepted and ready to land.Apr 9 2015, 6:30 AM
This revision was automatically updated to reflect the committed changes.