[PATCH] D81641: [SYCL][OpenMP] Implement thread-local storage restriction

Tue Jun 16 10:27:10 PDT 2020

Fznamznon added a comment.

In D81641#2088446 <https://reviews.llvm.org/D81641#2088446>, @jdoerfert wrote:

> OpenMP has the same restriction (no surprise I guess). Thanks for the ping!
>
> I think we do not emit diagnosis right now: https://godbolt.org/z/srDkXZ
>  I think we also should diagnose this the same way, though it might be beyond the scope of this patch: https://godbolt.org/z/rRZFi4

Alright, now the first case that you pointed is diagnosed as well.
The second case is not diagnosed at all and I agree that it seems beyond the scope of this patch, because it looks like using of `#pragma omp task untied` doesn't trigger emission of deferred diagnostics.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81641