Re: [C++ PATCH] Add CLEANUP_POINT_EXPRs around OMP_PARALLEL/TASK/FOR if needed (PR c++/51669)

2012-01-02 Thread Jason Merrill
On 01/02/2012 05:49 PM, Richard Henderson wrote: On 01/03/2012 09:10 AM, Jakub Jelinek wrote: Attached are two different patches, the first one puts the CLEANUP_POINT_EXPR around the whole OMP_PARALLEL etc. stmt, the second one wraps the individual clause expressions into CLEANUP_POINT_EXPR. Bo

Re: [C++ PATCH] Add CLEANUP_POINT_EXPRs around OMP_PARALLEL/TASK/FOR if needed (PR c++/51669)

2012-01-02 Thread Richard Henderson
On 01/03/2012 09:10 AM, Jakub Jelinek wrote: > Attached are two different patches, the first one puts the > CLEANUP_POINT_EXPR around the whole OMP_PARALLEL etc. stmt, the second > one wraps the individual clause expressions into CLEANUP_POINT_EXPR. > > Both patches have been bootstrapped/regteste

[C++ PATCH] Add CLEANUP_POINT_EXPRs around OMP_PARALLEL/TASK/FOR if needed (PR c++/51669)

2012-01-02 Thread Jakub Jelinek
Hi! The f1 function in the testcase fails newly starting with http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181332 because there is no CLEANUP_POINT_EXPR around OMP_PARALLEL/OMP_TASK/OMP_FOR whose IF/FINAL/NUM_THREADS/SCHEDULE clause expression might need some cleanups. But as the testcase show