[PATCH] D87667: [Support/Path] Add path::is_absolute_gnu

Thu Sep 24 03:40:45 PDT 2020

tinti added a comment.

This patch added an issue in some builds:

  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                   ^

See [1].

The fix is to use std::make_tuple.

Fixed by Mikael [2].

[1] http://lab.llvm.org:8011/builders/clang-cmake-x86_64-sde-avx512-linux/builds/45180
[2] https://github.com/llvm/llvm-project/commit/a1217620a87f66616c15e869d56783ba18e51b12

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87667