stella.stamenova added a comment.
I saw this test fail in our testing as well. I think the matches are too broad
and I suspect this will fail intermittently on the bots too causing confusion:
Command Output (stderr):
--
/vstsdrive/_work/10/s/llvm/tools/clang/test/Driver/print-multi-direct
chrib added a comment.
Re-commit patch at r341547 with modified test case to use an explicit root tree
for libraries
Repository:
rC Clang
https://reviews.llvm.org/D51354
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.o
Hello all,
Is there a way to see how the patch looked like after my commit and
before the revert from the git mirror ?
I cannot reproduce the test failure from the bots in my local build,
(which is based on the the git mirror). but I noticed that a similar
behavior however occurs when the test
timshen added a comment.
> The test fails on my system like so:
I also observed the same failure.
Bots also fail:
http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules/builds/19509/steps/check-all/logs/FAIL%3A%20Clang%3A%3Aprint-multi-directory.c
I'm going to revert this patch.
thakis added a comment.
The test fails on my system like so:
--
FAIL: Clang :: Driver/print-multi-directory.c (4696 of 13174)
TEST 'Clang :: Driver/print-multi-directory.c' FAILED
Script:
--
: 'RUN: at line 1';
/u
This revision was automatically updated to reflect the committed changes.
Closed by commit rC341373: Fix the -print-multi-directory flag to print the
selected multilib. (authored by chrib, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D51354?vs=162846&id=163835#toc
Reposito
jroelofs accepted this revision.
jroelofs added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks for fixing this!
Repository:
rC Clang
https://reviews.llvm.org/D51354
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
chrib added a comment.
https://bugs.llvm.org/show_bug.cgi?id=21360
Repository:
rC Clang
https://reviews.llvm.org/D51354
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
chrib created this revision.
Herald added subscribers: cfe-commits, srhines.
Fix -print-multi-directory to print the selected multilib
Repository:
rC Clang
https://reviews.llvm.org/D51354
Files:
include/clang/Driver/ToolChain.h
lib/Driver/Driver.cpp
lib/Driver/ToolChains/Linux.cpp
te
