[PATCH] D128260: [Fortran] Ignore whitespace in FCVS test results

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 23 05:06:21 PDT 2022


awarzynski added a comment.

In D128260#3604020 <https://reviews.llvm.org/D128260#3604020>, @kiranchandramohan wrote:

> And is it possible to have multiple outputs for comparison as @clementval mentioned?

I'm not too fond of the idea - that would require more bookkeeping and code to maintain. My preference would be to avoid this. Unless llvm-test-suite already has some logic for this (I'm not aware of such).

> One thing that we could do is to only enable whitespace for the 2 tests that use list-directed output.

You meant "disable" , right?

Either approach sounds fine to me - I would also be happy with updating FM905 and FM907.


Repository:
  rT test-suite

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

https://reviews.llvm.org/D128260



More information about the llvm-commits mailing list