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

Tue Jun 28 03:05:31 PDT 2022

awarzynski accepted this revision.
awarzynski added a comment.
This revision is now accepted and ready to land.

In D128260#3614645 <https://reviews.llvm.org/D128260#3614645>, @rovka wrote:

> No change, trying to convince the pre-merge to apply the patch...

How confident are you that it should work? I mean, perhaps the pre-merge CI has not been configured correctly for llvm-test-suite? I've just tried `arc patch D128260` and it worked fine for me locally. This looks like the pre-merge CI issue. Perhaps create a bug <https://github.com/google/llvm-premerge-checks/issues/new?assignees=&labels=bug&template=bug_report.md&title=buildkite%20build%20diff-checks%20112318>?

To me all this makes sense and hence approving as is, thanks! AFAIK, the only buildbots running these tests are managed by Linaro, so you will know if something is broken :)

Repository:
  rT test-suite

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

https://reviews.llvm.org/D128260