Phabricator pre-merge check failed due to seemingly unrelated issues

Hi,

I uploaded a bug fix in phabricator: βš™ D141909 Avoid removing useful loop metadata when striping debug info
The pre-merge check failed on x64 debian due to some tests timeout:

Timed Out Tests (7):
Clang :: CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/non-overloaded/vloxseg.c
Clang :: CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/non-overloaded/vluxseg.c
Clang :: CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/overloaded/vloxseg.c
Clang :: CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/overloaded/vluxseg.c
libFuzzer :: fuzzer-leak.test
libFuzzer :: minimize_crash.test
libFuzzer :: value-profile-load.test

I try running the RISCV tests locally and they pass, but they do take somewhat long (a few minutes in Debug build). But that is the case with or without my change.

Anyone knows what is the problem and what should I do?

It’s pretty common for a small set of long running tests to hit timeouts in pre-commit (and I’ve seen it happen on under resourced post commit buildbots too).

Seems pretty unlikely you broke anything here so I would ignore the failures. If the post commit buildbots come up with the same failures, then you can revert and investigate.

When I see some failures that look unrelated to my changes, I just check neighboring PRs to make sure they fail the same way :slight_smile:

2 Likes

Thanks for the info!