rjmccall accepted this revision. rjmccall added a comment. In https://reviews.llvm.org/D40275#937010, @tra wrote:
> In https://reviews.llvm.org/D40275#933253, @tra wrote: > > > @rjmccall : are you OK with this approach? If VLA is not supported by the > > target, CUDA is handled as a special case so it can emit deferred diag, > > OpenMP reports an error only if shouldDiagnoseTargetSupportFromOpenMP() > > allows it, and everything else does so unconditionally. > > > @rjmccall : ping. Sorry for the delay; I took Thanksgiving week off. Yes, I think this patch is fine now, thanks. https://reviews.llvm.org/D40275 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
