[PATCH] D113076: [flang] Add a semantics test for co_sum

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 11:13:54 PDT 2021


kiranchandramohan requested changes to this revision.
kiranchandramohan added inline comments.
This revision now requires changes to proceed.


================
Comment at: flang/test/Semantics/collectives01.f90:1
+! RUN: %S/test_errors.sh %s %t %flang_fc1
+! REQUIRES: shell
----------------
Thanks @rouson for these tests. Recently we switched to using python (test_errors.py) instead of shell scripts (test_errors.sh) so that these tests run on Windows as well.

Please see some of the other tests for an example. (https://github.com/llvm/llvm-project/blob/main/flang/test/Semantics/critical01.f90)

Please switch to using that otherwise, the tests will fail.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113076/new/

https://reviews.llvm.org/D113076



More information about the llvm-commits mailing list