[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-09-10 Thread Stella Stamenova via Phabricator via cfe-commits
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): --

[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-09-06 Thread Christian Bruel via Phabricator via cfe-commits
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

Re: [PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-09-05 Thread Christian BRUEL via cfe-commits
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

[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-09-04 Thread Tim Shen via Phabricator via cfe-commits
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] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-09-04 Thread Nico Weber via Phabricator via cfe-commits
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';

[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-09-04 Thread Christian Bruel via Phabricator via cfe-commits
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=163835#toc

[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-08-28 Thread Jonathan Roelofs via Phabricator via cfe-commits
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

[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-08-28 Thread Christian Bruel via Phabricator via cfe-commits
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

[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

2018-08-28 Thread Christian Bruel via Phabricator via 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