[PATCH] D43233: Use EXPECT_FALSE instead of EXPECT_EQ(false, ...

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 06:00:25 PST 2018


chill created this revision.
chill added a reviewer: gkistanova.

Commit https://reviews.llvm.org/rL324489 added

  EXPECT_EQ(false, N->isUnsigned());

which older GCC versions dislike for some reason. Anyway, it looks like the proper GTest way
is to use `EXPECT_FALSE`, etc.


Repository:
  rL LLVM

https://reviews.llvm.org/D43233

Files:
  unittests/IR/MetadataTest.cpp


Index: unittests/IR/MetadataTest.cpp
===================================================================
--- unittests/IR/MetadataTest.cpp
+++ unittests/IR/MetadataTest.cpp
@@ -981,7 +981,7 @@
   auto *N = DIEnumerator::get(Context, 7, false, "name");
   EXPECT_EQ(dwarf::DW_TAG_enumerator, N->getTag());
   EXPECT_EQ(7, N->getValue());
-  EXPECT_EQ(false, N->isUnsigned());
+  EXPECT_FALSE(N->isUnsigned());
   EXPECT_EQ("name", N->getName());
   EXPECT_EQ(N, DIEnumerator::get(Context, 7, false, "name"));
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43233.134027.patch
Type: text/x-patch
Size: 510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180213/e1f7326a/attachment.bin>


More information about the llvm-commits mailing list