[PATCH] D61742: [Driver][Windows] Add dependent lib argument for profile instr generate
Nico Weber via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 27 17:04:17 PDT 2019
thakis added a comment.
Hello, this embeds an absolute path into the generated .obj file, which means the output now is no longer deterministic (since it depends on the absolute path to clang_rt.profile-x86_64.lib). This means the output will be different on different machines, which breaks things like caching in distributed builds and whatnot.
I can't see an obvious way to save this patch either, since you won't know at compile time what the CWD will be at link time.
As-is, this breaks Chromium's deterministic builds. ("Only" the coverage builds, and they don't yet check for determinism, which is why it took us a while to notice.)
CHANGES SINCE LAST ACTION
More information about the cfe-commits