[PATCH] D103119: [scudo] Get rid of initLinkerInitialized

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 14 11:24:14 PDT 2021


cryptoad added a comment.

In D103119#2807183 <https://reviews.llvm.org/D103119#2807183>, @yroux wrote:

> Hi,
>
> This change introduced a regression on ARMv7 Thumb bots which wasn't noticed due to other issues.
>
> logs can be seen here: https://lab.llvm.org/buildbot/#/builders/26/builds/2251
>
> The 3 tests fail with a bus error raised from tsd_shared.h line 49

Hey, do you have more details about this? (I was on vacation last few days).
https://github.com/llvm/llvm-project/blob/main/compiler-rt/lib/scudo/standalone/tsd_shared.h#L49 is just the end of a block. Looking at the output of the failing tests in the bots, it's just empty.
I am unclear as to how this would only fail on arm thumb as this part is not arch specific.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103119



More information about the llvm-commits mailing list