This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Move flang OpenMP semantic tests under one single directory
ClosedPublic

Authored by peixin on Jun 10 2022, 12:26 PM.

Details

Summary

To be consistent with OpenACC and will help find the tests in separate
directories for OpenMP and Fortran semantic checks.

Diff Detail

Event Timeline

peixin created this revision.Jun 10 2022, 12:26 PM
Herald added a project: Restricted Project. · View Herald Transcript
peixin requested review of this revision.Jun 10 2022, 12:26 PM

Nice. LGTM.

If you chose to skip the following it is fine. Otherwise consider these too, but not a strong opinion.
flang/test/Semantics/canondo05.f90:! RUN: %flang_fc1 -fopenmp -fdebug-unparse-with-symbols %s 2>&1 | FileCheck %s
flang/test/Semantics/canondo06.f90:! RUN: %flang_fc1 -fopenmp -fdebug-unparse-with-symbols %s 2>&1 | FileCheck %s
flang/test/Semantics/modfile47.f90:! RUN: %python %S/test_modfile.py %s %flang_fc1 -fopenmp

This revision is now accepted and ready to land.Jun 10 2022, 2:02 PM
peixin updated this revision to Diff 436090.Jun 10 2022, 7:02 PM

If you chose to skip the following it is fine. Otherwise consider these too, but not a strong opinion.
flang/test/Semantics/canondo05.f90:! RUN: %flang_fc1 -fopenmp -fdebug-unparse-with-symbols %s 2>&1 | FileCheck %s
flang/test/Semantics/canondo06.f90:! RUN: %flang_fc1 -fopenmp -fdebug-unparse-with-symbols %s 2>&1 | FileCheck %s
flang/test/Semantics/modfile47.f90:! RUN: %python %S/test_modfile.py %s %flang_fc1 -fopenmp

Thanks for the notice. Fixed.

flang/test/Semantics/modfile47.f90