This is an archive of the discontinued LLVM Phabricator instance.

Merge all the llvm-exegesis unit tests into a single binary
ClosedPublic

Authored by tstellar on Jul 21 2021, 10:33 AM.

Details

Summary

These tests access private symbols in the backends, so they cannot link
against libLLVM.so and must be statically linked. Linking these tests
can be slow and with debug builds the resulting binaries use a lot of
disk space.

By merging them into a single test binary means we now only need to
statically link 1 test instead of 6, which helps reduce the build
times and saves disk space.

Diff Detail

Event Timeline

tstellar created this revision.Jul 21 2021, 10:33 AM
tstellar requested review of this revision.Jul 21 2021, 10:33 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 21 2021, 10:33 AM
courbet accepted this revision.Jul 21 2021, 11:17 PM

Thanks for doing this.

There are a few formatting errors to fix.

This revision is now accepted and ready to land.Jul 21 2021, 11:17 PM
This revision was landed with ongoing or failed builds.Jul 26 2021, 10:08 PM
This revision was automatically updated to reflect the committed changes.