Blacklisted instrumenting _Balloc.

Blacklisted instrumenting _Balloc.

Increases number of spec2k tests that run successfully with ASan from 2 to 6.

BUG=https://bugs.chromium.org/p/nativeclient/issues/detail?id=4374
R=kschimpf@google.com

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=3f97afb1675218f1a4cd7f30fc4faa86fcbeea03

[tlively, 2016-07-07 21:12:16]

tlively@google.com changed reviewers:
+ kschimpf@google.com, mcgrathr@chromium.org, stichnot@chromium.org

[Karl, 2016-07-07 21:20:49]

lgtm

https://codereview.chromium.org/2128383002/diff/1/src/IceASanInstrumentation.cpp
File src/IceASanInstrumentation.cpp (right):

https://codereview.chromium.org/2128383002/diff/1/src/IceASanInstrumentation....
src/IceASanInstrumentation.cpp:66: return FuncName == "" ||
FuncBlackList.find(FuncName) == FuncBlackList.end();
Actually, a simpler test is:

    || FuncBlackList.count(FuncName) == 0;

[tlively, 2016-07-07 21:56:24]

Description was changed from

==========
Blacklisted instrumenting _Balloc.

Increases number of spec2k tests that run successfully with ASan from 2 to 6.

BUG=https://bugs.chromium.org/p/nativeclient/issues/detail?id=4374
==========

to

==========
Blacklisted instrumenting _Balloc.

Increases number of spec2k tests that run successfully with ASan from 2 to 6.

BUG=https://bugs.chromium.org/p/nativeclient/issues/detail?id=4374
R=kschimpf@google.com

Committed:
https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a...
==========

[tlively, 2016-07-07 21:56:26]

Committed patchset #2 (id:20001) manually as
3f97afb1675218f1a4cd7f30fc4faa86fcbeea03 (presubmit successful).

[tlively, 2016-07-08 16:56:21]

https://codereview.chromium.org/2128383002/diff/1/src/IceASanInstrumentation.cpp
File src/IceASanInstrumentation.cpp (right):

https://codereview.chromium.org/2128383002/diff/1/src/IceASanInstrumentation....
src/IceASanInstrumentation.cpp:66: return FuncName == "" ||
FuncBlackList.find(FuncName) == FuncBlackList.end();
On 2016/07/07 21:20:49, Karl wrote:
> Actually, a simpler test is:
> 
>     || FuncBlackList.count(FuncName) == 0;

Done.