Use /analyze:WX- instead of /WX- - avoids 3925 D9025 warnings.

When building with /analyze there are, inevitably, thousands of
warnings. Since going /analyze warning-free is a non-goal we can't
have warnings be errors. This was previously handled by overriding
the /WX setting with /WX- but this worked patchily and led to lots
of these D9025 warnings:

    cl : Command line warning D9025 : overriding '/WX' with '/WX-'

I just discovered that /analyze:WX- can be used to say that /analyze
warnings should not be treated as errors. This avoids contradicting
/WX and therefore avoids the 3925 D9025 warnings. This makes the raw
results much easier to look through.

This change necessitates updating the /analyze exclusions in libexif
and yasm - running /analyze on those projects causes internal
compiler errors.

BUG=427616
R=scottmg@chromium.org,thestig@chromium.org,dalecurtis@chromium.org

Review URL: https://codereview.chromium.org/1177153007

Cr-Original-Commit-Position: refs/heads/master@{#335578}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 8a019cccd65c94b3156ed57322d264ae04aa115e
1 file changed