This is an archive of the discontinued LLVM Phabricator instance.

Remove 'RUN: false' from Driver/response-file.c
ClosedPublic

Authored by krasimir on Jan 20 2017, 2:36 AM.

Details

Summary

It seems that rL292518 introduced a RUN: false, but the continuation rL292545
forgot to remove it back.

This has flown under the radar, because it's a long test and doesn't get
executed by default during sanity testing.

To test:
$ cd llvm_build
$ ./bin/llvm-lit --param run_long_tests=true tools/clang/test/Driver/response-file.c

@rsmith: have a look if this change is OK please.

Event Timeline

krasimir created this revision.Jan 20 2017, 2:36 AM
krasimir edited the summary of this revision. (Show Details)Jan 20 2017, 2:38 AM
krasimir added a reviewer: bkramer.
krasimir added subscribers: rsmith, cfe-commits.
bkramer accepted this revision.Jan 20 2017, 3:08 AM

lg. I assume the test passes with the RUN: false removed?

This revision is now accepted and ready to land.Jan 20 2017, 3:08 AM

@bkramer: yes, it passes with RUN: false removed

This revision was automatically updated to reflect the committed changes.