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

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 00:49:07 PDT 2022


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>?

I commented on https://github.com/google/llvm-premerge-checks/issues/329

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



More information about the llvm-commits mailing list