[PATCH] D41128: [scudo] Adding a public Scudo interface

Sun Dec 31 15:58:17 PST 2017

Hahnfeld added a comment.

In https://reviews.llvm.org/D41128#965450, @cryptoad wrote:

> In https://reviews.llvm.org/D41128#964426, @Hahnfeld wrote:
>
> > The added test has been disabled on armhf in https://reviews.llvm.org/rCRT320665 because it fails a bot: http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15-full-sh/builds/2952/steps/ninja%20check%202/logs/FAIL%3A%20Scudo-armhf%3A%3A%20interface.cpp
> >
> > I see the exact same error on x86_64 CentOS 7, did you find out what the problem is? I'm happy to assist if you have problems to reproduce the problem...
>
>
> I haven't been able to repro the issue. If you have a configuration or some insight as to what the failure is, please let me know!

Any normal build on CentOS 7 failed reliably. I've submitted https://reviews.llvm.org/D41649 which solves the problem for me, I'll see if this also fixes the failure on ARM...

Side note: Do I understand this correctly that scudo will only get active on an intercepted `malloc` and exceeded limits without a `malloc` have no effect? That's a huge restriction, probably most applications in the HPC world first allocate all structures and only start touching the memory afterwards...

Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41128