On Mon, Oct 12, 2026 at 3:42 PM Marco Elver <[email protected]> wrote: > > +Rust code is instrumented as well on architectures whose Rust target supports > +it (currently only x86_64; see ``CONFIG_HAVE_KCSAN_RUSTC``).
Maybe we can omit the x86_64 bit, to avoid maintaining the list in two places, although it is convenient to know in the docs (but risks getting out of date, of course). > + Note that ``rustc`` rejects linking crates built with and without KCSAN > + instrumentation, so ``KCSAN_SANITIZE`` and ``KCSAN_SANITIZE_<target>.o`` > + cannot currently be used for Rust code. Would `-Cunsafe-allow-abi-mismatch=sanitizer` work? Not a blocker for the patch (and in fact I don't know if we would like to actually support it even if it works), but perhaps in the future we can relax it using that. (If it doesn't work for some reason, and we want it, then we can ask upstream for an "escape hatch" as we usually do, since things like this should normally have one...) > +# The minimum supported rustc's LLVM supports all of these options. Just to double-check: does this refer to the LLVM version of the upstream `rustc` binary (LLVM 19.1.7) or the actual minimum they support in that minimum `rustc` version (LLVM 18 major)? I think all the options exist for a long time, so it doesn't matter, but just to confirm... In any case, this sounds all reasonable: Acked-by: Miguel Ojeda <[email protected]> Thanks! Cheers, Miguel
