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

Francis Visoiu Mistrih via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 09:43:34 PDT 2019


Author: thegameg
Date: Wed Oct 16 09:43:34 2019
New Revision: 375015

URL: http://llvm.org/viewvc/llvm-project?rev=375015&view=rev
Log:
[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

Modified:
    llvm/trunk/unittests/Remarks/YAMLRemarksParsingTest.cpp

Modified: llvm/trunk/unittests/Remarks/YAMLRemarksParsingTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Remarks/YAMLRemarksParsingTest.cpp?rev=375015&r1=375014&r2=375015&view=diff
==============================================================================
--- llvm/trunk/unittests/Remarks/YAMLRemarksParsingTest.cpp (original)
+++ llvm/trunk/unittests/Remarks/YAMLRemarksParsingTest.cpp Wed Oct 16 09:43:34 2019
@@ -82,11 +82,13 @@ void parseExpectErrorMeta(StringRef Buf,
 
   // 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) {




More information about the llvm-commits mailing list