https://reviews.llvm.org/D121809 causes the build bot failure, add the REQUIRES: asserts to fix it.
Details
Details
Diff Detail
Diff Detail
- Repository
- rLLD LLVM Linker
Event Timeline
Comment Actions
Sorry for the patch abuse, the pre-merge build bot didn't catch the failures, like https://lab.llvm.org/buildbot/#/builders/124/builds/3948 and https://lab.llvm.org/buildbot/#/builders/67/builds/6214. Hopefully, this patch will fix it.
Comment Actions
lgtm
No worries, this is not an uncommon situation. Also, you don't need a patch review for a small bot fix like this. Generally just go ahead and commit these types of fixes. Alternatively, if the fix is more complicated than something like this, you could revert the original patch and upload the fixed version of the patch for re-review.