Dimitar Dimitrov <dimi...@dinux.eu> writes: > 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.
OK, great. I went ahead and applied the patch. Thanks, Richard
