cchen marked an inline comment as done. cchen added inline comments.
================ Comment at: clang/lib/Sema/SemaOpenMP.cpp:16624 + if (IsPointer && !AllowAnotherPtr) + SemaRef.Diag(ELoc, diag::err_omp_section_length_undefined) << true; + else ---------------- ABataev wrote: > Better to use integer value as selectors, not boolean. The selector for `err_omp_section_length_undefined` is a bool value. (true for unknown bound false for not a array type, so always be true here). Do you mean that I need to create a new kind of diagnosis message here and use integer as selectors? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79972/new/ https://reviews.llvm.org/D79972 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
