Hi H.J., > On Tue, Oct 24, 2017 at 2:33 AM, Uros Bizjak <ubiz...@gmail.com> wrote: >> On Tue, Oct 24, 2017 at 10:39 AM, Tsimbalist, Igor V >> <igor.v.tsimbal...@intel.com> wrote: >>> OK. >> >> +/* { dg-final { scan-assembler-times "endbr32" 2 { target ia32 } } } */ >> +/* { dg-final { scan-assembler-times "endbr64" 2 { target { ! ia32 } } } } >> */ >> >> I think we can only check for {\mendbr} in the testcases. There are >> already plenty of testcases that check for the correct instruction. >> >> Otherwise, LGTM >> > > This is what I checked in.
the cet-sjlj-5.c test, like cet-sjlj-3.c, didn't account for an empty USER_LABEL_PREFIX. Fixed as the previous one, installed. Rainer -- ----------------------------------------------------------------------------- Rainer Orth, Center for Biotechnology, Bielefeld University 2017-10-25 Rainer Orth <r...@cebitec.uni-bielefeld.de> * gcc.target/i386/cet-sjlj-5.c: Allow for emtpy user label prefix in setjmp call.
# HG changeset patch # Parent bca0754daa2001dcf85a01cd8be2aad3cbd48fcd Fix gcc.target/i386/cet-sjlj-5.c on Solaris diff --git a/gcc/testsuite/gcc.target/i386/cet-sjlj-5.c b/gcc/testsuite/gcc.target/i386/cet-sjlj-5.c --- a/gcc/testsuite/gcc.target/i386/cet-sjlj-5.c +++ b/gcc/testsuite/gcc.target/i386/cet-sjlj-5.c @@ -2,7 +2,7 @@ /* { dg-options "-O -fcf-protection -mcet" } */ /* { dg-final { scan-assembler-times "endbr32" 2 { target ia32 } } } */ /* { dg-final { scan-assembler-times "endbr64" 2 { target { ! ia32 } } } } */ -/* { dg-final { scan-assembler-times "call _setjmp" 1 } } */ +/* { dg-final { scan-assembler-times "call _?setjmp" 1 } } */ /* { dg-final { scan-assembler-times "call longjmp" 1 } } */ #include <stdio.h>