efriedma added inline comments.
================ Comment at: clang/lib/Sema/SemaType.cpp:2278 ArraySize, &SizeVal, Diagnoser, - (S.LangOpts.GNUMode || S.LangOpts.OpenCL) ? Sema::AllowFold - : Sema::NoFold); + S.LangOpts.OpenCL ? Sema::AllowFold : Sema::NoFold); if (Diagnoser.IsVLA) ---------------- efriedma wrote: > We probably want some note to explain why OpenCL is special here... if you > can't figure out easily, please leave a FIXME. Oh, I see you posted a separate patch for that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89523/new/ https://reviews.llvm.org/D89523 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits