[clang-tidy] For checker `readability-misleading-indentation` update tests.
authorAndi-Bogdan Postelnicu <abpostelnicu@me.com>
Thu, 9 Jan 2020 08:00:05 +0000 (10:00 +0200)
committerAndi-Bogdan Postelnicu <abpostelnicu@me.com>
Thu, 9 Jan 2020 11:52:26 +0000 (13:52 +0200)
Summary: In D72333 we've introduced support for `if constexpr` but the test for uninstantiated template was not ready to land on windows platform since this target uses `-fdelayed-template-parsing` by default. This patch addresses this by passing `-fno-delayed-template-parsing` to the test.

Reviewers: JonasToth

Subscribers: xazax.hun, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D72438

clang-tools-extra/test/clang-tidy/checkers/readability-misleading-indentation.cpp

index c3bd33d..aea0618 100644 (file)
@@ -1,4 +1,4 @@
-// RUN: %check_clang_tidy %s readability-misleading-indentation %t
+// RUN: %check_clang_tidy %s readability-misleading-indentation %t -- -- -fno-delayed-template-parsing
 
 void foo1();
 void foo2();
@@ -169,6 +169,17 @@ void mustFailNonTemplate() {
 }
 
 template<bool b>
+void mustFailNoInsta() {
+  if constexpr (b) {
+    foo1();
+  }
+    else {
+      foo2();
+      // CHECK-MESSAGES: :[[@LINE-2]]:5: warning: different indentation for 'if' and corresponding 'else' [readability-misleading-indentation]
+  }
+}
+
+template<bool b>
 void mustPassNoInsta() {
   if constexpr (b) {
     foo1();