[PATCH] D91037: [clang-tidy] Fix crash in bugprone-redundant-branch-condition on ExprWithCleanups

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 11 01:39:36 PST 2020


njames93 added inline comments.


================
Comment at: clang-tools-extra/test/clang-tidy/checkers/bugprone-redundant-branch-condition.cpp:1092
+      // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: redundant condition 'isSet' [bugprone-redundant-branch-condition]
+      // CHECK-FIXES: {{^\ *$}}
+    }
----------------
This line isn't really doing anything. `CHECK-FIXES` just runs FileCheck on the whole file after clang tidy has applied any fixes. You need to explicitly say what you are expecting it to be. 


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

https://reviews.llvm.org/D91037



More information about the cfe-commits mailing list