Fixed instruction corruption bug for multiple returns.

Fixed instruction corruption bug for multiple returns.

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=a7e5a951381aeb5390c460b810cd2b01e888ee4a

[tlively, 2016-07-06 19:47:04]

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

[tlively, 2016-07-06 19:47:05]

https://codereview.chromium.org/2128643002/diff/1/pydir/szbuild.py
File pydir/szbuild.py (right):

https://codereview.chromium.org/2128643002/diff/1/pydir/szbuild.py#newcode310
pydir/szbuild.py:310: args.sz_args.append('-fsanitize-address')
This redundancy check is necessary because szbuild_spec2k.py runs this code path
multiple times.

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

https://codereview.chromium.org/2128643002/diff/1/src/IceASanInstrumentation....
src/IceASanInstrumentation.cpp:305: HasTailCall, IsTargetHelperCall);
Since Insts are linked list nodes, they cannot be inserted multiple times, and
copies must be made.

[Karl, 2016-07-06 20:21:33]

lgtm

[tlively, 2016-07-06 20:46:15]

Description was changed from

==========
Fixed instruction corruption bug for multiple returns.

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

to

==========
Fixed instruction corruption bug for multiple returns.

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-06 20:46:17]

Committed patchset #1 (id:1) manually as
a7e5a951381aeb5390c460b810cd2b01e888ee4a (presubmit successful).