ellis added a comment.
I've just published https://reviews.llvm.org/D157664, so you'll want to rebase
ontop of it if it lands soon. I would also like to see some more tests added to
`instrprof-merge-error.c` to make sure two different binaries can't merge
profiles together with `--debug-info-correlate`. I was thinking the test would
be something like this.
// RUN: %clang_pgogen -o %t/a -g -mllvm --debug-info-correlate -mllvm
--disable-vp=true %t/main.c
// RUN: %clang_pgogen -o %t/b -g -mllvm --debug-info-correlate -mllvm
--disable-vp=true %t/main.c %t/foo.c
// RUN: env LLVM_PROFILE_FILE=%t/default_%m.profdata %run %t/a
// This next line should fail to merge because the counter sections have
different sizes
// RUN: env LLVM_PROFILE_FILE=%t/default_%m.profdata %run %t/b
//--- main.c
int main() { return 0; }
//--- foo.c
int foo() { return 4; }
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157632/new/
https://reviews.llvm.org/D157632
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
