[all-commits] [llvm/llvm-project] d421a6: [flang][driver] Remove `flang -help` tests (#89504)

Andrzej Warzyński via All-commits all-commits at lists.llvm.org
Wed Apr 24 11:47:49 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d421a6c9ef1dd286832fce3b4f955d4ba99ce192
      https://github.com/llvm/llvm-project/commit/d421a6c9ef1dd286832fce3b4f955d4ba99ce192
  Author: Andrzej Warzyński <andrzej.warzynski at arm.com>
  Date:   2024-04-24 (Wed, 24 Apr 2024)

  Changed paths:
    R flang/test/Driver/driver-help-hidden.f90
    M flang/test/Driver/driver-help.f90

  Log Message:
  -----------
  [flang][driver] Remove `flang -help` tests (#89504)

These tests were introduced when the Flang driver was originally
incepted. Back then, `flang-new` only supported a handful of basic
options (e.g. `-v`, `-help`, `-E`) and those `-help` tests were used to
guard against Clang options "leaking" into `flang-new -help`.

With the introduction of the concept of "visibility" (see
https://reviews.llvm.org/D157837 and https://reviews.llvm.org/D157151),
these tests are effectively redundant. Also, with the number of flags
growing, these tests are getting rather brittle.



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


More information about the All-commits mailing list