================
@@ -14192,11 +14194,22 @@ class SequenceChecker : public 
ConstEvaluatedExprVisitor<SequenceChecker> {
     if (OtherKind == UK_Use)
       std::swap(Mod, ModOrUse);
 
+    unsigned DiagID = 0;
+    switch (WarnKind) {
+    case WK_UseAndMod:
+      DiagID = diag::warn_unsequenced_mod_use;
----------------
shafik wrote:

Your tests are only covering one of these diagnostics, we need tests that cover 
all three. IF we don't then we are likely to introduce regressions or perhaps 
it is likely there is uncaught bug.

https://github.com/llvm/llvm-project/pull/180955
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to