jsjodin wrote:

> I'm afraid I'm not much help reviewing the OpenMP aspects of this, but the 
> CIR handling and refactoring look good.

Yes, there's not really any new functionality mostly refactoring. So I didn't 
bother adding any of the OMP rereivewers, but I can do that. The only 
difference is that I added support for
`\#pragma omp target parallel`
Compared to 
```
\#pragma omp target {
\#pragma omp parallel {
}
}
```
In the first case clang combines things into a single node, so we have to have 
a case for it.

https://github.com/llvm/llvm-project/pull/207019
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to