On Wed, Mar 9, 2022 at 2:44 PM David Laight <David.Laight@xxxxxxxxxx> wrote: > > From: Kees Cook > > Sent: 09 March 2022 22:10 > > > > Unfortunately, Clang did not have support for "sp" as a global register > > definition, and was crashing after the addition of current_stack_pointer. > > This has been fixed in Clang 15, but earlier Clang versions need to > ^^ 14 Yeah, with that fixed: Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> Once clang-14 is the minimally supported version, we can go back to removing the inline asm for everyone. > > avoid this code, so add a versioned test and revert back to the > > open-coded asm instances. Fixes Clang build error: > > > > fatal error: error in backend: Invalid register name global variable > > David > > - > Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK > Registration No: 1397386 (Wales) > -- Thanks, ~Nick Desaulniers
