This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, PR60189, Cilk+] Fix for ICE with incorrect Cilk_sync usage
- From: Jeff Law <law at redhat dot com>
- To: "Zamyatin, Igor" <igor dot zamyatin at intel dot com>, "'Jason Merrill'" <jason at redhat dot com>
- Cc: "'GCC Patches (gcc-patches at gcc dot gnu dot org)'" <gcc-patches at gcc dot gnu dot org>, "Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>
- Date: Tue, 20 May 2014 10:38:57 -0600
- Subject: Re: [PATCH, PR60189, Cilk+] Fix for ICE with incorrect Cilk_sync usage
- Authentication-results: sourceware.org; auth=none
- References: <0EFAB2BDD0F67E4FB6CCC8B9F87D7569429DBFEF at IRSMSX103 dot ger dot corp dot intel dot com> <0EFAB2BDD0F67E4FB6CCC8B9F87D7569429DC138 at IRSMSX103 dot ger dot corp dot intel dot com> <20140410142731 dot GF1817 at tucnak dot redhat dot com> <5348027F dot 1050407 at redhat dot com> <0EFAB2BDD0F67E4FB6CCC8B9F87D7569429E0533 at IRSMSX103 dot ger dot corp dot intel dot com> <534B6054 dot 1070800 at redhat dot com> <0EFAB2BDD0F67E4FB6CCC8B9F87D7569429EE571 at IRSMSX101 dot ger dot corp dot intel dot com> <534C1F71 dot 5040508 at redhat dot com> <0EFAB2BDD0F67E4FB6CCC8B9F87D7569429FC4CC at IRSMSX103 dot ger dot corp dot intel dot com> <0EFAB2BDD0F67E4FB6CCC8B9F87D756942A58124 at IRSMSX101 dot ger dot corp dot intel dot com> <0EFAB2BDD0F67E4FB6CCC8B9F87D7569564312ED at IRSMSX103 dot ger dot corp dot intel dot com>
On 05/20/14 08:10, Zamyatin, Igor wrote:
Please look then on the following patch.
Regtested successfully on x86_64.
Is it ok for trunk and 4.9?
gcc/cp/ChangeLog:
2014-05-20 Igor Zamyatin <igor.zamyatin@intel.com>
PR c/60189
* parser.c (cp_parser_postfix_expression): Move handling of cilk_sync
from here to...
(cp_parser_statement): ...here. Make sure only semicolon can go after
Cilk_sync.
gcc/testsuite/ChangeLog:
2014-05-20 Igor Zamyatin <igor.zamyatin@intel.com>
PR c++/60189
* c-c++-common/cilk-plus/CK/invalid_sync.cÑ: New test.
This is fine for both the trunk and the 4.9 branch.
Thanks,
Jeff