This is an archive of the discontinued LLVM Phabricator instance.

[fuzzer] Reduce size of large.test
ClosedPublic

Authored by nikic on Apr 22 2022, 1:23 AM.

Details

Summary

This halves the size of LargeTest, dropping time to compile this file locally from 14s to 5.5s. Hopefully this will also fix the persistent timeouts in pre-merge checks.

Diff Detail

Event Timeline

nikic created this revision.Apr 22 2022, 1:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2022, 1:23 AM
nikic requested review of this revision.Apr 22 2022, 1:23 AM
nikic added inline comments.
compiler-rt/test/fuzzer/LargeTest.cpp
28

Note that the E macro was already unused previously.

vitalybuka accepted this revision.May 9 2022, 10:53 AM
This revision is now accepted and ready to land.May 9 2022, 10:53 AM
fhahn added a subscriber: fhahn.May 10 2022, 1:02 AM

Looks like the test is still regularly timing out. Might be good to land the patch, as unfortunately there has been no progress with respect to increasing the test timeout.

This revision was landed with ongoing or failed builds.May 10 2022, 1:20 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMay 10 2022, 1:20 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript