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

[Diana Picus via Phabricator via flang-commits]

rovka added a comment.

In D128260#3614974 <https://reviews.llvm.org/D128260#3614974>, @awarzynski wrote:

> 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 :)

Yep, I'll keep an eye out :)

> **EDIT** Same issue with patch application: https://reviews.llvm.org/D126453 (this is quite recent)


Repository:
  rT test-suite

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

https://reviews.llvm.org/D128260