On Wed, Aug 14, 2024 at 12:40 AM <andrey.konovalov@xxxxxxxxx> wrote: > > Signed-off-by: Andrey Konovalov <andreyknvl@xxxxxxxxx> It is easier to read now, and indeed GCC 5.1+ and LLVM 13+ both support the flags, so `CFLAGS_KASAN_SHADOW` can't be empty. > +# First, enable -fsanitize=kernel-address together with providing the shadow > +# mapping offset, as for GCC, -fasan-shadow-offset fails without -fsanitize > +# (GCC accepts the shadow mapping offset via -fasan-shadow-offset instead of > +# a normal --param). Instead of ifdef-checking the compiler, rely on cc-option. I guess "a normal --param" means here that it is the usual way to tweak the rest of the KASAN parameters, right? > +# Now, add other parameters enabled in a similar way with GCC and Clang. I think the "with" sounds strange, but I am not a native speaker. Perhaps "in a similar way with" -> "similarly in both"? Thanks! Cheers, Miguel
