[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