[Openmp-commits] [PATCH] D58454: [OpenMP][OMPT] Fix locking testcases for 32 bit architectures

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld added a comment.

In D58454#1405654 <https://reviews.llvm.org/D58454#1405654>, @mgorny wrote:

> In D58454#1405643 <https://reviews.llvm.org/D58454#1405643>, @Hahnfeld wrote:
>
> > In D58454#1405621 <https://reviews.llvm.org/D58454#1405621>, @krytarowski wrote:
> >
> > > This broke the NetBSD buildbot: http://lab.llvm.org:8011/builders/lldb-amd64-ninja-netbsd8/builds/19082
> >
> >
> > This should be fixed in rOMP354572 <https://reviews.llvm.org/rOMP354572>. As a bot owner, could you please make sure that the bot lists the commits to `openmp`? Otherwise it's hard to detect broken bots without emails.
>
>
> If I only knew how, I'd have fixed that long ago.


If I had to bet, that's because `lldb-amd64-ninja-netbsd8` declares itself to be an `lldb` builder: https://github.com/llvm/llvm-zorg/blob/master/buildbot/osuosl/master/config/builders.py#L976
As a result it will only be triggered by commits to `llvm`, `cfe` and `lldb`: https://github.com/llvm/llvm-zorg/blob/master/buildbot/osuosl/master/master.cfg#L194
And if buildbot doesn't know that the bot magically updates and tests `openmp`, it probably doesn't add the relevant commits and so on.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58454