This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Add bolt-runtime requirement to tests
ClosedPublic

Authored by urnathan on Apr 19 2023, 1:02 PM.

Details

Summary

These tests rely on the X86 runtime, add the REQUIRES.

IIUC 'REQUIRES: a && b' is equivalent to 'REQUIRES: a,b', I changed the one instance of the former to the latter and added the bolt-runtime requirement.

I have further patches that allow one to disable the runtime even when x86 is the host -- these tests start failing in that situation.

Diff Detail

Event Timeline

urnathan created this revision.Apr 19 2023, 1:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 19 2023, 1:02 PM
urnathan requested review of this revision.Apr 19 2023, 1:02 PM
This revision is now accepted and ready to land.Apr 19 2023, 1:40 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 19 2023, 4:42 PM