Yes, that's almost certainly related, and probably due to:
https://codereview.chromium.org/2940873002
We should test that and probably revert that change and re-roll GN if we can't fix it relatively easily.
Owner: dpranke@chromium.org Status: Started (was: Available)
I tested this by just adding a generic #define to //build/config/compiler and re-running analyze, and can confirm that it incorrectly returned "no compile necessary". And, reverting the above CL reverted to the default "give up and compile everything" behavior.
In the interest of not leaving things broken I'm going to revert the change, roll GN, and then we can look at relanding a fixed version.
This should be (and seems to be) fixed now. https://chromium-review.googlesource.com/c/619991 has a link to a patch that modified a base compiler config and analyze worked correctly in a tryjob on linux_chromium_rel_ng.
Of course, we still want to land a working version of the GN patch, but that's not this bug.
Comment 1 by thakis@chromium.org
, Aug 15 2017