Fznamznon added a comment.
Herald added a subscriber: yaxunl.

In D74387#1950593 <https://reviews.llvm.org/D74387#1950593>, @jdoerfert wrote:

> This is needed for OpenMP as well. Does it make sense to include it in this 
> patch or in another one?


I thought OpenMP already has diagnostics for unsupported types (at least 
looking into this commit 
https://github.com/llvm/llvm-project/commit/123ad1969171d0b22d0c5d0ec23468586c4d8fa7).
 Am I wrong?
The diagnostic which I'm implementing here is stricter than existing OpenMP 
diagnostic, the main goal is do not emit unsupported type at all. Does OpenMP 
need such restriction as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74387



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to