Run the integration tests against the Clang instrumented DLL.

This make the integration tests also use the Clang version of the test
DLL.

None of the tests has been modified.

Review-Url: https://codereview.chromium.org/2983783002
Committed: https://github.com/google/syzygy/commit/9de882bf08f7a2af66b6ce233da3644b9c988a6a

[Sébastien Marchand, 2017-08-30 00:43:09]

Description was changed from

==========
[DO NOT COMMIT] Prototype for the Clang integration tests.
==========

to

==========
This make the integration tests also use the Clang version of the test
DLL.

None of the tests has been modified.
==========

[Sébastien Marchand, 2017-08-30 00:43:09]

sebmarchand@chromium.org changed reviewers:
+ chrisha@chromium.org

[Sébastien Marchand, 2017-08-30 00:43:55]

PTAL

[chrisha, 2017-08-30 16:55:32]

lgtm!

https://codereview.chromium.org/2983783002/diff/240001/syzygy/integration_tes...
File syzygy/integration_tests/asan_interceptors_tests.h (right):

https://codereview.chromium.org/2983783002/diff/240001/syzygy/integration_tes...
syzygy/integration_tests/asan_interceptors_tests.h:42: return *location;
You don't need the try/except for the Clang version. The body of the function
could be simpler in this case. (Like you did with the NonInterceptedWrite
function.)

[Sébastien Marchand, 2017-08-31 16:41:28]

The CQ bit was checked by sebmarchand@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-31 16:41:30]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sébastien Marchand, 2017-08-31 17:46:09]

On 2017/08/30 16:55:32, chrisha wrote:
> lgtm!
> 
>
https://codereview.chromium.org/2983783002/diff/240001/syzygy/integration_tes...
> File syzygy/integration_tests/asan_interceptors_tests.h (right):
> 
>
https://codereview.chromium.org/2983783002/diff/240001/syzygy/integration_tes...
> syzygy/integration_tests/asan_interceptors_tests.h:42: return *location;
> You don't need the try/except for the Clang version. The body of the function
> could be simpler in this case. (Like you did with the NonInterceptedWrite
> function.)

Thanks, committing.

[commit-bot: I haz the power, 2017-08-31 17:46:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-31 17:46:35]

Dry run: This issue passed the CQ dry run.

[Sébastien Marchand, 2017-08-31 17:49:16]

The CQ bit was checked by sebmarchand@chromium.org

[Sébastien Marchand, 2017-08-31 17:49:16]

The patchset sent to the CQ was uploaded after l-g-t-m from chrisha@chromium.org
Link to the patchset: https://codereview.chromium.org/2983783002/#ps260001
(title: "Address Chris comment.")

[commit-bot: I haz the power, 2017-08-31 17:49:23]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-08-31 17:49:27]

CQ is committing da patch.

Bot data: {"patchset_id": 260001, "attempt_start_ts": 1504201756139670,
"parent_rev": "7b2e1976baae45930725f0b59252a23c9f78c4a5", "commit_rev":
"9de882bf08f7a2af66b6ce233da3644b9c988a6a"}

[commit-bot: I haz the power, 2017-08-31 17:49:31]

Description was changed from

==========
This make the integration tests also use the Clang version of the test
DLL.

None of the tests has been modified.
==========

to

==========
This make the integration tests also use the Clang version of the test
DLL.

None of the tests has been modified.

Review-Url: https://codereview.chromium.org/2983783002
Committed:
https://github.com/google/syzygy/commit/9de882bf08f7a2af66b6ce233da3644b9c988a6a
==========

[commit-bot: I haz the power, 2017-08-31 17:49:33]

Committed patchset #14 (id:260001) as
https://github.com/google/syzygy/commit/9de882bf08f7a2af66b6ce233da3644b9c988a6a