[PATCH] D131830: Clang Support for taskwait nowait clause

[Sunil K via Phabricator via llvm-commits]

koops added a comment.

Johannes Doerfert,
Most of your comments are related to avoiding having the __kmpc_omp_taskwait_51 and instead introduce a "nowait" related parameter in the __kmpc_omp_taskwait itself. I have explained earlier that __kmpc_omp_taskwait_51( ) is a  placeholder for introducing new code, at a later stage. The scheduling of tasks might change in the runtime due to "nowait" clause. I have done this with the sole purpose of having backward compatibility. If you feel that is not needed then I will remove the new code and modify the Code Generation also according to your comments.

Thank you for your detailed comments.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131830/new/

https://reviews.llvm.org/D131830