DescriptionUse /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
Committed: https://crrev.com/8a019cccd65c94b3156ed57322d264ae04aa115e
Cr-Commit-Position: refs/heads/master@{#335578}
Patch Set 1 #Patch Set 2 : Pulling latest code. #Patch Set 3 : Adding back libexif exclusion from /analyze #
Messages
Total messages: 12 (4 generated)
|