Apparently tools/determinism/remove_buld_metadata.py currently runs on bots after a build and modifies build artifacts. That makes it hard to understand when this step breaks something, and modifying built artifacts in-place seems like it has other undesirable properties as well.

Instead, this should run as part of a regular build, either during the link step, or the link step should write binaries to foo.pre.exe and then the determinism step should take the pre.exe and write the final .exe (if determinism is explicitly enabled, on platforms where this is needed.)

This came up in  issue 729059 .