From: Francis Visoiu Mistrih <francisvm@yahoo.com>
Date: Wed, 16 Oct 2019 16:43:34 +0000 (+0000)
Subject: [Remarks] Fix warning for ambigous `else` behind EXPECT macro
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a55be6b48f453e6355bcc2f73a9c431e6a0c7870;p=llvm

[Remarks] Fix warning for ambigous `else` behind EXPECT macro

http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/31902/steps/ninja%20check%201/logs/stdio

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@375015 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/unittests/Remarks/YAMLRemarksParsingTest.cpp b/unittests/Remarks/YAMLRemarksParsingTest.cpp
index 5ab7b21367d..f20cdb5e6bb 100644
--- a/unittests/Remarks/YAMLRemarksParsingTest.cpp
+++ b/unittests/Remarks/YAMLRemarksParsingTest.cpp
@@ -82,11 +82,13 @@ void parseExpectErrorMeta(StringRef Buf, const char *Error, CmpType Cmp,
 
   // Use a case insensitive comparision due to case differences in error strings
   // for different OSs.
-  if (Cmp == CmpType::Equal)
+  if (Cmp == CmpType::Equal) {
     EXPECT_EQ(StringRef(Stream.str()).lower(), StringRef(Error).lower());
+  }
 
-  if (Cmp == CmpType::Contains)
+  if (Cmp == CmpType::Contains) {
     EXPECT_TRUE(StringRef(Stream.str()).contains(StringRef(Error)));
+  }
 }
 
 TEST(YAMLRemarks, ParsingEmpty) {