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

Christian BRUEL via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 5 07:11:08 PDT 2018


Understood, The test worked for me because I have both 64 bit and 32 bit libs installed on my default sysroot. Otherwise might fail indeed.

Will amend the test with an explicit sysroot, 


> -----Original Message-----
> From: Tim Shen via Phabricator [mailto:reviews at reviews.llvm.org]
> Sent: Wednesday, September 05, 2018 12:12 AM
> To: Christian BRUEL <christian.bruel at st.com>; jroelofs at jroelofs.com
> Cc: timshen at google.com; srhines at google.com; cfe-commits at lists.llvm.org
> Subject: [PATCH] D51354: Fix the -print-multi-directory flag to print the
> selected multilib.
> 
> 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.
> 
> 
> Repository:
>   rC Clang
> 
> https://reviews.llvm.org/D51354
> 
> 



More information about the cfe-commits mailing list