[PATCH] D82470: [OpenMP][IRBuilder] Support nested parallel regions

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 28 10:47:54 PDT 2020


jdoerfert added a comment.

In D82470#2116581 <https://reviews.llvm.org/D82470#2116581>, @SouraVX wrote:

> Overall this seems great Thanks! I have one minor concern:
>  This patch seems to do 2 things: Support for nested parallel regions(which was crashing earlier) and some infrastructure change(introducing AllocBuilder..).
>  I'm not sure of this, but is it possible to separate these as 2( or more) patches ? 1 for Nested parallel region support and other patch as a infrastructure change ?


I'll try.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82470





More information about the cfe-commits mailing list