Dimitar Dimitrov <dimi...@dinux.eu> writes: > I have tested this fix on x86_64 host, and found no regression in the C > and C++ testsuites. I'm marking this patch as RFC simply because I don't > have experience with other architectures, and I don't have a setup to > test all architectures supported by GCC. > > gcc/ChangeLog: > > 2018-12-07 Dimitar Dimitrov <dimi...@dinux.eu> > > * cfgexpand.c (asm_clobber_reg_is_valid): Also produce > error when stack pointer is clobbered. > (expand_asm_stmt): Refactor clobber check in separate function. > > gcc/testsuite/ChangeLog: > > 2018-12-07 Dimitar Dimitrov <dimi...@dinux.eu> > > * gcc.target/i386/pr52813.c: New test. > > Signed-off-by: Dimitar Dimitrov <dimi...@dinux.eu>
LGTM. Do you have a copyright assignment on file? 'Fraid this is probably big enough to need one. Thanks, Richard
