[PATCH] Implement variable-sized alloca instrumentation (take 2).

Maxim Ostapenko m.ostapenko at partner.samsung.com
Tue May 12 10:45:11 PDT 2015


Konstantin, thank you for review!

In http://reviews.llvm.org/D7098#169544, @kcc wrote:

> LGTM
>  Looks simple enough.
>  This is still off by default, right? Did you test it on something huge?


Yes, it's off by default. I've managed to build Firefox with dynamic allocas instrumentation enabled, is it big enough? I'm also found a "bug" there and I'm trying to understand if it's a real bug (not just unpoisoning error or something like that).


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7098

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list