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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

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

> After the reverting of changes, putting the default in the switch case as discussed in the comments earlier.

No, do not put the default: switch, instead you need explicitly list all other enumeric values, like

  case 1:
  case 2:
    break;


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

https://reviews.llvm.org/D123235