On 15.09.2025 18:56, Saman Dehghan wrote: > This patch updates the LLVM profile raw format in > `xen/common/coverage/llvm.c` from version 4 to version 10, > enabling compatibility with LLVM versions 19 and 20. > While the patch supports only one version: > 1. It seems better to support one version than no version -- > the current profile version 4 is not compatible with > LLVM version 11 or later
Indeed. > 2. The patch could be extended to support multiple > LLVM profile versions, e.g., from 5 to 10 Would certainly be nice, perhaps as a follow-on. > The llvm-cov toolchain, with its Source-based Code Coverage, > offers two substantial advantages over gcov: > - More accurate coverage reporting when compiler optimizations > are enabled, ensuring better analysis of optimized code. > - Better tracking of coverage across inlined function boundaries, > critical for complex control flows in Xen. > > Overall, this change would enhance Xen's code coverage analysis > capabilities by leveraging the latest LLVM toolchain improvements, > particularly for safety-critical hypervisor code. > > The patch modifies only `xen/common/coverage/llvm.c`, > maintaining API compatibility while enabling modern toolchain support. > Testing was performed with LLVM 19 and 20 to confirm functionality. > > --- What's missing in any event is your Signed-off-by:. Jan
