ABataev added a comment.

In D123235#4656464 <https://reviews.llvm.org/D123235#4656464>, @koops wrote:

> In clang/lib/AST/OpenMPClause.cpp,
>
>   OMPClauseWithPreInit::get(const OMPClause *C) {
>   switch(C->getClauseKind()) {
>   case OMPC_schedule:
>   ....
>   default:
>       break;
>   }
>
> It is not possible to list down all possible ``OpenMPClauseKind``` types in 
> the switch. Hence they have used a default even in other methods. I am 
> following the same logic.

Why not possible?


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

https://reviews.llvm.org/D123235

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

Reply via email to