[PATCH] D35854: Fix double destruction of objects when OpenMP construct is canceled

2017-07-27 Thread Erich Keane via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL309288: Fix double destruction of objects when OpenMP construct is canceled (authored by erichkeane). Changed prior to commit: https://reviews.llvm.org/D35854?vs=108153&id=108489#toc Repository: rL L

[PATCH] D35854: Fix double destruction of objects when OpenMP construct is canceled

2017-07-26 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D35854 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D35854: Fix double destruction of objects when OpenMP construct is canceled

2017-07-25 Thread Mike Rice via Phabricator via cfe-commits
mikerice created this revision. When an omp for loop is canceled the constructed objects are being destructed twice. It looks like the desired code is: { Obj o; If (cancelled) branch-through-cleanups to cancel.exit. } [cleanups] cancel.exit: __kmpc_for_static_fini br cancel.cont (*)