This is an archive of the discontinued LLVM Phabricator instance.

[flang] Port test_any.sh tests to FileCheck
ClosedPublic

Authored by richard.barton.arm on Jun 19 2020, 2:37 AM.

Details

Summary

Port the remaining tests which only require mechanical changes.

  • Delete old RUN lines
  • Replace:
EXEC: ${F18} ... | ${FileCheck} ...
with
RUN: %f18 .. | FileCheck ...
  • Prepend RUN line with not when it is expected to fail

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald Transcript
DavidTruby added inline comments.Jun 20 2020, 8:12 AM
flang/test/Semantics/canondo05.f90
1–2

What does this XXXEXEC line do here? I don't think it's something FileCheck will understand

richard.barton.arm edited the summary of this revision. (Show Details)Jun 22 2020, 7:45 AM
richard.barton.arm added a project: Restricted Project.

Reinstate previously crippled EXEC line

richard.barton.arm marked 2 inline comments as done.Jun 22 2020, 8:51 AM
richard.barton.arm added inline comments.
flang/test/Semantics/canondo05.f90
1–2

This was a test_any.sh 'RUN' line, so named EXEC so that lit would not try to run it, that was broken. Looking at the logs it is likely the original test was added before F18 could parse the OpenMP directives inside. Simple change to reinstate this as a ported RUN line.

schweitz accepted this revision.Jun 26 2020, 9:00 AM
schweitz added a subscriber: schweitz.

Did you remove the test_any.sh script in one of these?

Thanks for porting these.

This revision is now accepted and ready to land.Jun 26 2020, 9:00 AM
richard.barton.arm marked an inline comment as done.Jun 26 2020, 10:25 AM

I'll do it in this one when I commit.

Thanks for the review!

This revision was automatically updated to reflect the committed changes.
flang/test/Semantics/label13.f90