koops updated this revision to Diff 558051. koops added a comment. To avoid build error in ppc64 adding a "default" to switch statement. This is similar to the way it is currently handled in
OMPClauseWithPreInit::get() { switch(C->getClauseKind()) { case ... .... default: } CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123235/new/ https://reviews.llvm.org/D123235 Files: clang/lib/Basic/OpenMPKinds.cpp Index: clang/lib/Basic/OpenMPKinds.cpp =================================================================== --- clang/lib/Basic/OpenMPKinds.cpp +++ clang/lib/Basic/OpenMPKinds.cpp @@ -442,12 +442,13 @@ case OMPC_fail: { OpenMPClauseKind CK = static_cast<OpenMPClauseKind>(Type); switch (CK) { - case OMPC_unknown: - return "unknown"; #define OPENMP_ATOMIC_FAIL_MODIFIER(Name) \ case OMPC_##Name: \ return #Name; #include "clang/Basic/OpenMPKinds.def" + case OMPC_unknown: + default: + return "unknown"; } llvm_unreachable("Invalid OpenMP 'fail' clause modifier"); }
Index: clang/lib/Basic/OpenMPKinds.cpp =================================================================== --- clang/lib/Basic/OpenMPKinds.cpp +++ clang/lib/Basic/OpenMPKinds.cpp @@ -442,12 +442,13 @@ case OMPC_fail: { OpenMPClauseKind CK = static_cast<OpenMPClauseKind>(Type); switch (CK) { - case OMPC_unknown: - return "unknown"; #define OPENMP_ATOMIC_FAIL_MODIFIER(Name) \ case OMPC_##Name: \ return #Name; #include "clang/Basic/OpenMPKinds.def" + case OMPC_unknown: + default: + return "unknown"; } llvm_unreachable("Invalid OpenMP 'fail' clause modifier"); }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits