This is an archive of the discontinued LLVM Phabricator instance.

[OMPT] Remove test output from source tree
ClosedPublic

Authored by Hahnfeld on Feb 9 2019, 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

Event Timeline

Hahnfeld created this revision.Feb 9 2019, 9:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 9 2019, 9:17 AM
protze.joachim accepted this revision.Feb 11 2019, 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.Feb 11 2019, 1:00 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2019, 8:15 AM