Page MenuHomePhabricator

[OMPT] Remove test output from source tree
ClosedPublic

Authored by Hahnfeld on Sat, Feb 9, 9:17 AM.

Details

Summary

%s refers to the test file in the source tree. This was accidentally added in
r351197 / 2b46d30 ("[OMPT] Second chunk of final OMPT 5.0 interface updates").

Diff Detail

Repository
rL LLVM

Event Timeline

Hahnfeld created this revision.Sat, Feb 9, 9:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptSat, Feb 9, 9:17 AM
protze.joachim accepted this revision.Mon, Feb 11, 1:00 AM

LGTM, thanks for catching this!
The output is written to the build dir anyways:

runtime/test/lit.cfg:127

config.substitutions.append(("FileCheck", "tee %%t.out | %s" % config.test_filecheck))

This revision is now accepted and ready to land.Mon, Feb 11, 1:00 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMon, Feb 11, 8:15 AM