[all-commits] [llvm/llvm-project] 93dca9: [flang][test] Fix semantics tests with respect to ...

Emil Kieri via All-commits all-commits at lists.llvm.org
Thu Aug 18 10:16:54 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 93dca9fbeea6c1f83223a621710eaf01c06350db
      https://github.com/llvm/llvm-project/commit/93dca9fbeea6c1f83223a621710eaf01c06350db
  Author: Emil Kieri <j.emil.kieri at gmail.com>
  Date:   2022-08-18 (Thu, 18 Aug 2022)

  Changed paths:
    M flang/test/Semantics/bindings01.f90
    M flang/test/Semantics/block-data01.f90
    M flang/test/Semantics/call03.f90
    M flang/test/Semantics/case01.f90
    M flang/test/Semantics/common-blocks.f90
    M flang/test/Semantics/data06.f90
    M flang/test/Semantics/dosemantics02.f90
    M flang/test/Semantics/dosemantics12.f90
    M flang/test/Semantics/forall01.f90
    M flang/test/Semantics/int-literals.f90
    M flang/test/Semantics/resolve11.f90
    M flang/test/Semantics/resolve31.f90
    M flang/test/Semantics/resolve35.f90
    M flang/test/Semantics/resolve42.f90
    M flang/test/Semantics/resolve59.f90
    M flang/test/Semantics/resolve85.f90

  Log Message:
  -----------
  [flang][test] Fix semantics tests with respect to warnings

Make tests expect the (correctly) emitted warnings using the WARNING
directive. This directive is non-functional now, but will be recognised
by test_errors.py when D125804 is landed. This patch is a preparation
for D125804.

For most tests, we add missing WARNING directives for emitted warnings,
but there are exceptions:

 - for int-literals.f90 and resolve31.f90 we pass -pedantic to the
   frontend driver, so that the expected warnings are actually emitted.

 - for block-data01.f90 and resolve42.f90 we change the tests so that
   warnings, which appear unintentional, are not emitted. While testing
   the warning in question (padding added for alignment in common block)
   would be desired, that is beyond the scope of this patch. This
   warning is target-dependent.

Reviewed By: PeteSteinfeld

Differential Revision: https://reviews.llvm.org/D131987




More information about the All-commits mailing list