This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Add NewPM testing to CFI tests
ClosedPublic

Authored by tejohnson on Jul 17 2018, 9:50 AM.

Details

Summary

Executes both LTO and ThinLTO CFI tests an additional time using the new
pass manager. I only bothered to add with gold and not lld as testing
with one linker should be sufficient. I didn't add for APPLE or WIN32
since I don't have a way to test those.

Depends on D49429.

Diff Detail

Repository
rL LLVM

Event Timeline

tejohnson created this revision.Jul 17 2018, 9:50 AM
Herald added subscribers: Restricted Project, dexonsmith, delcypher and 3 others. · View Herald TranscriptJul 17 2018, 9:50 AM
pcc accepted this revision.Jul 18 2018, 2:59 PM

LGTM

I didn't add for APPLE or WIN32 since I don't have a way to test those.

It doesn't seem possible to activate the new pass manager in COFF lld or with the old LTO API, so it doesn't look like there's anything to test there right now anyway.

This revision is now accepted and ready to land.Jul 18 2018, 2:59 PM
This revision was automatically updated to reflect the committed changes.