[all-commits] [llvm/llvm-project] 3e1786: [Flang] Make all tests work with lit's internal shell

[Aiden Grossman via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3e17864e76f593b8558cbb37f96d883fd9d60fc7
      https://github.com/llvm/llvm-project/commit/3e17864e76f593b8558cbb37f96d883fd9d60fc7
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-08-29 (Fri, 29 Aug 2025)

  Changed paths:
    M flang/docs/FlangDriver.md
    M flang/test/Driver/color-diagnostics-parse.f90
    M flang/test/Driver/color-diagnostics-scan.f
    M flang/test/Driver/color-diagnostics-sema.f90
    M flang/test/Driver/color-diagnostics.f90
    M flang/test/Lower/Intrinsics/selected_int_kind.f90
    M flang/test/Lower/Intrinsics/selected_real_kind.f90
    M flang/test/Lower/OpenMP/parallel-firstprivate-clause-scalar.f90
    M flang/test/Lower/OpenMP/parallel-private-clause-str.f90
    M flang/test/Lower/OpenMP/parallel-private-clause.f90
    M flang/test/Lower/OpenMP/threadprivate-integer-different-kinds.f90
    M flang/test/Lower/OpenMP/wsloop-variable.f90
    M flang/test/Semantics/OpenMP/do07.f90
    M flang/test/Semantics/OpenMP/invalid-branch.f90
    M flang/test/Semantics/OpenMP/nontemporal.f90
    M flang/test/Semantics/missing_newline.f90

  Log Message:
  -----------
  [Flang] Make all tests work with lit's internal shell

This patch removes REQUIRES: shell lines and updates the tests to not
need a shell, or adds REQUIRES lines for Linux where relevant. A lot of
these seem to have gotten introduced in
58c3f20bbf51f454ffb38ddb700a1d5ad374dc7b, but many are no longer
relevant as these tests no longer use shell scripts.

There was one test (missing_newline.f90) that needed to be modified to
work. Lit doesn't recognize -ne correctly for echo and the options need
to be split into separate flags.

Reviewers: clementval, Meinersbur, kiranchandramohan, klausler

Reviewed By: clementval

Pull Request: https://github.com/llvm/llvm-project/pull/156115



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications