[cfe-dev] Updating googletest to v1.10.0

Bal√°zs Benics via cfe-dev cfe-dev at lists.llvm.org
Fri Apr 24 02:17:14 PDT 2020

Hi all,

On a Phabricator review (https://reviews.llvm.org/D78704) the `GTEST_SKIP`
mentioned as a way to express requirements for a test. In that patch, I
to skip the test if clang was not built with Z3.

I think this `GTEST_SKIP` macro express the intention much better than the
following code that I saw throughout the codebase:

  do { \
    if (isUnsupportedOSOrEnvironment()) \
      return; \
  } while (0);

There are several places where we might benefit from the `GTEST_SKIP` macro:
- llvm/unittests/ExecutionEngine/MCJIT/MCJITTestAPICommon.h:29
- llvm/unittests/Support/ThreadPool.cpp:81
- llvm/unittests/Support/MemoryTest.cpp:89

It is possible that the upgrade would introduce other benefits as well -
I'm not aware of.

Should we upgrade googletest to version 1.10.0?

/CC Chandler

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20200424/622e9e85/attachment.html>

More information about the cfe-dev mailing list