[all-commits] [llvm/llvm-project] a12176: [unittests] Use std::make_tuple to make some toolc...

mikaelholmen via All-commits all-commits at lists.llvm.org
Thu Sep 24 02:26:28 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a1217620a87f66616c15e869d56783ba18e51b12
      https://github.com/llvm/llvm-project/commit/a1217620a87f66616c15e869d56783ba18e51b12
  Author: Mikael Holmen <mikael.holmen at ericsson.com>
  Date:   2020-09-24 (Thu, 24 Sep 2020)

  Changed paths:
    M llvm/unittests/Support/Path.cpp

  Log Message:
  -----------
  [unittests] Use std::make_tuple to make some toolchains happy again

My toolchain stopped working (LLVM 8.0, libstdc++ 5.4.0) after 577adda:

06:25:37 ../unittests/Support/Path.cpp:91:7: error: chosen constructor is explicit in copy-initialization
06:25:37       {"", false, false},  {"/", true, true},      {"/foo", true, true},
06:25:37       ^~~~~~~~~~~~~~~~~~
06:25:37 /proj/flexasic/app/llvm/8.0/bin/../lib/gcc/x86_64-unknown-linux-gnu/5.4.0/../../../../include/c++/5.4.0/tuple:479:19: note: explicit constructor declared here
06:25:37         constexpr tuple(_UElements&&... __elements)
06:25:37                   ^

This commit adds explicit calls to std::make_tuple to work around
the problem.




More information about the All-commits mailing list