[PATCH] D108942: [compiler-rt] Add more diagnostic to InstrProfError

Gulfem Savrun Yeniceri via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 9 13:13:12 PST 2021


gulfem added a comment.

In D108942#3119615 <https://reviews.llvm.org/D108942#3119615>, @dyung wrote:

> In D108942#3119594 <https://reviews.llvm.org/D108942#3119594>, @RKSimon wrote:
>
>> @gulfem This is failing on some buildbots: https://lab.llvm.org/buildbot/#/builders/109/builds/25948 please can you take a look?
>
> The test seems to require zlib which means either that dependency needs to be removed, or a "REQUIRES: zlib" needs to be added.

@RKSimon and @dyung,
Thanks for reporting, and I have already pushed a change that removed the dependency to zlib on those tests:
https://github.com/llvm/llvm-project/commit/126e7611c70ca41782aa851c2bec132607eb8127
After the fix, bot turned green.
https://lab.llvm.org/buildbot/#/builders/109/builds/25973


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108942



More information about the llvm-commits mailing list