uabelho added a comment. Btw, a bit confusing (to me at least) to reopen this review since it's already landed.
================ Comment at: clang/lib/Basic/OpenMPKinds.cpp:450 + case OMPC_unknown: + default: + return "unknown"; ---------------- Adding "default:" here silences the warning. But looks like @ABataev commented on something similar earlier and said that's not recommended? 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
