[PATCH] D138846: MC/DC in LLVM Source-Based Code Coverage: LLVM back-end and compiler-rt

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 22 07:30:19 PST 2023


hans added a comment.

> I just saw @glandium's earlier comment:
>
>> Code built with older versions of LLVM (e.g. rust) and running with the updated runtime crash at startup with this change.
>
> This is the exact same issue we encountered. Because there is a profile format change, it's expected to update both clang and rustc to use the same version of llvm in order for it to work.

Thanks for figuring this out!

Would it be possible to somehow make profile format mismatches a linker error instead of a hard-to-debug runtime problem? For example could the instrumentation depend on some symbol in the runtime whose name includes a version number?

I think the ASan (and maybe other sanitizer) instrumentations do this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138846/new/

https://reviews.llvm.org/D138846



More information about the cfe-commits mailing list