On понеделник, 10 декември 2018 г. 11:21:53 EET Richard Sandiford wrote: > 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. Yes, I have copyright assignment.
Regards, Dimitar