[flang-commits] [PATCH] D149116: [Fortran/gfortran] Enable "compile" tests from the gfortran torture test suite

Fri May 19 07:03:15 PDT 2023

luporl added a comment.

In D149116#4352863 <https://reviews.llvm.org/D149116#4352863>, @tarunprabhu wrote:

> In D149116#4343386 <https://reviews.llvm.org/D149116#4343386>, @luporl wrote:
>
>> But I am not familiar with llvm-test-suite. Do you have an idea of
>> how much effort would be required?
>
> Yes, I agree that modifying the test suite would be a better idea in the long run. However, I, too, am not very familiar with the llvm-test-suite so I am not sure just how much work it would be. I wanted to just get something working for now so we have a larger suite of tests that the community can use.
>
> I could look into the modifying the test-suite itself, but I would like to prioritize getting most of the gfortran test-suite working first.

I'm ok with getting most of the gfortran test-suite working first.

What happened to Fortran/gfortran/compile-save-diags.cmake file? It seems it was deleted in the last diff.

================
Comment at: Fortran/gfortran/torture/compile/CMakeLists.txt:159-162
+list(APPEND UnsupportedTests "")
+list(APPEND UnimplementedTests "")
+list(APPEND SkippedTests "")
+list(APPEND FailingTests "${Failing}")
----------------
tarunprabhu wrote:
> luporl wrote:
> > Is it expected that UnsupportedTests/UnimplementedTests/SkippedTests will receive entries when more compile tests are enabled?
> Yes, it is possible if gfortran adds more tests to this list and if they exercise features that we don't support, for instance. The idea is for these tests to be periodically synced with gfortran so we can take advantage of any more tests that they add.
Ok, it makes sense.

Repository:
  rT test-suite

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

https://reviews.llvm.org/D149116