[PATCH] D60644: [compiler-rt][builtins][sanitizers] Guard test cases with macros to run when specific version of GLIBC is detected

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 15 20:46:52 PDT 2019


hubert.reinterpretcast added a comment.

In D60644#1466135 <https://reviews.llvm.org/D60644#1466135>, @rupprecht wrote:

> Part of me thinks this is bad -- if the version of glibc is old, then the host toolchain should just upgrade it. glibc 2.23 was released in 2016-02-19, so I think using something older than that is on the border of being called "not a modern c++ toolchain".


We talk of needing a "modern C++ toolchain" for building the project; however, in the case of `compiler_rt_logb_test`, we are talking not of the build environment, but rather the supporting libraries on a system where compiler-rt may be deployed. Perhaps the tests are currently set up to run in the build environment, but is not a stretch to imagine that the tests can be run as part of an installer for deploying the build.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60644/new/

https://reviews.llvm.org/D60644





More information about the llvm-commits mailing list