This is an archive of the discontinued LLVM Phabricator instance.

Test default compare in global status page
ClosedPublic

Authored by thopre on Sep 7 2020, 7:16 AM.

Details

Reviewers
cmatthews
Summary

Make run for profile report older so that the global status page uses
another run.

Event Timeline

thopre requested review of this revision.Sep 7 2020, 7:16 AM
thopre created this revision.
thopre planned changes to this revision.Sep 10 2020, 2:43 AM

Forgot to test without the change and realized today the test still passes so it's not testing the change.

thopre updated this revision to Diff 291240.Sep 11 2020, 9:06 AM
thopre retitled this revision from Add test for default compare in global status page to Test default compare in global status page.
thopre edited the summary of this revision. (Show Details)

Fix testcase to fail without D69056 and pass with

cmatthews accepted this revision.Oct 2 2020, 1:01 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Oct 2 2020, 1:01 PM
thopre closed this revision.Oct 3 2020, 12:49 AM

LGTM. Thanks!

Thanks for the review! Landed as 22f39a4879da0908ac0abb0697aa5e73a9771934