[PATCH] D123235: [OpenMP] atomic compare fail : Parser & AST support

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 8 08:14:31 PST 2023


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



More information about the cfe-commits mailing list